Dapper CVE-2010-3880, inet_diag: Make sure we actually run the same bytecode we audited

[Tim Gardner]

> On 02/09/2011 01:40 PM, Brad Figg wrote:
>>>> struct inet_diag_entry entry;
>>>> - struct rtattr *bc = (struct rtattr *)(r + 1);
>>>> + struct rtattr *bc = nlmsg_find_attr(cb->nlh, sizeof(*r),
>>>> + INET_DIAG_REQ_BYTECODE);
>>
>> In the other patches this was changed to:
>> const struct nlattr *bc = nlmsg_find_attr(cb->nlh,
>> sizeof(*r),
>> INET_DIAG_REQ_BYTECODE);
>>
>>
>
> I dropped 'const' because the prototype for nlmsg_find_attr() doesn't
> call for it in 2.6.15.
>

OK, I've corrected the type change from rtattr to nlattr.

rtg
-- 
Tim Gardner tim.gardner at canonical.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-inet_diag-Make-sure-we-actually-run-the-same-bytecod.patch
Type: text/x-patch
Size: 3109 bytes
Desc: not available
URL: <https://lists.ubuntu.com/archives/kernel-team/attachments/20110209/52709116/attachment.bin>