Re: [PATCH] SELinux: null dereference in error path

2005-03-01 Thread Folkert van Heusden
> >>The 'bad' label will call function that unconditionally dereferences > >>the NULL pointer. > >>Found by the Coverity tool > >>Signed-off-by: Alexander Nyberg <[EMAIL PROTECTED]> > >Signed-off-by: James Morris <[EMAIL PROTECTED]> > Err, isn't it "Acked-by:"?? I thought "Signed-off-by:" was only

Re: [PATCH] SELinux: null dereference in error path

2005-03-01 Thread Stephen Smalley
On Tue, 2005-03-01 at 01:32 +0100, Alexander Nyberg wrote: > The 'bad' label will call function that unconditionally dereferences > the NULL pointer. > > Found by the Coverity tool > > Signed-off-by: Alexander Nyberg <[EMAIL PROTECTED]> Acked-by: Stephen Smalley <[EMAIL PROTECTED]> -- Stephen

Re: [PATCH] SELinux: null dereference in error path

2005-02-28 Thread Andrew Morton
Kyle Moffett <[EMAIL PROTECTED]> wrote: > > Err, isn't it "Acked-by:"?? Yes, I usually change it to Acked-by when it's obvious. Signed-off-by: "I worked on this patch" Acked-by: "Looks good" Tested-by: "You're kidding" - To unsubscribe from this list: send the line "unsubscribe linux-

Re: [PATCH] SELinux: null dereference in error path

2005-02-28 Thread Kyle Moffett
On Feb 28, 2005, at 23:11, James Morris wrote: On Tue, 1 Mar 2005, Alexander Nyberg wrote: The 'bad' label will call function that unconditionally dereferences the NULL pointer. Found by the Coverity tool Signed-off-by: Alexander Nyberg <[EMAIL PROTECTED]> Signed-off-by: James Morris <[EMAIL PROTEC

Re: [PATCH] SELinux: null dereference in error path

2005-02-28 Thread James Morris
On Tue, 1 Mar 2005, Alexander Nyberg wrote: > The 'bad' label will call function that unconditionally dereferences > the NULL pointer. > > Found by the Coverity tool > > Signed-off-by: Alexander Nyberg <[EMAIL PROTECTED]> Signed-off-by: James Morris <[EMAIL PROTECTED]> -- James Morris <[EMAI

[PATCH] SELinux: null dereference in error path

2005-02-28 Thread Alexander Nyberg
The 'bad' label will call function that unconditionally dereferences the NULL pointer. Found by the Coverity tool Signed-off-by: Alexander Nyberg <[EMAIL PROTECTED]> = security/selinux/ss/policydb.c 1.16 vs edited = --- 1.16/security/selinux/ss/policydb.c 2005-01-15 23:01:45 +01:00 +++ e