[PATCH] s4-ranged_result.c: Fix memory context for ranged attributes handling

Kamen Mazdrashki kamenim at samba.org
Wed Dec 1 02:08:12 MST 2010


On Tue, Nov 30, 2010 at 23:08, Andrew Bartlett <abartlet at samba.org> wrote:
> On Tue, 2010-11-30 at 19:04 +0200, Kamen Mazdrashki wrote:
>
> Aside from the extra semi-colon:
>
>> -       /* LDB_REPLY_ENTRY */
>> +       /* LDB_REPLY_ENTRY */;
>
oups :)
It seems my keyboard has it's own opinion how code should be written :)
Thanks for catching this!

> This looks good.  The original code clearly allocated a pointer that was
> potentially the second element in an array.
>
> Nice catch, both of you!
>
> (if autobuild cooperates, please push).
>
Thanks!
I wonder if we should wait a little bit so Zak to make a test first
to exploit this behavior (other way would be for him to write the test
on a branch without this fix)?


-- 
CU,
Kamen


More information about the samba-technical mailing list