Re: [PATCH 4/4] cifs: verify lengths of QueryAllEAs reply

On Mon, Jan 11, 2010 at 7:00 PM, Jeff Layton <[email protected]> wrote:

> On Mon, 11 Jan 2010 15:57:44 -0600
> Steve French <[email protected]> wrote:
>
> > > +               }
> > > +
> > > +               name_len = temp_fea->name_len;
> > > +               value_len = le16_to_cpu(temp_fea->value_len);
> > > +               if (temp_ptr + name_len + value_len > end_of_smb) {
>                                    ^^^^^^^^
>                        erm...I think I forgot to figure in the null
>                        terminator here on the name. I'll respin and
>                        resend. In case it's not obvious, please be
>                        sure to sanity check my pointer math in this
>                        patch :)
>
>
ok ... :)


-- 
Thanks,

Steve

_______________________________________________
linux-cifs-client mailing list
[email protected]
https://lists.samba.org/mailman/listinfo/linux-cifs-client