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

[folkert@vanheusden.com (Folkert van Heusden)]

[-- Attachment #1: Type: text/plain, Size: 1102 bytes --]

> >>The 'bad' label will call function that unconditionally dereferences
> >>the NULL pointer.
> >>Found by the Coverity tool
> >>Signed-off-by: Alexander Nyberg <alexn@dsv.su.se>
> >Signed-off-by: James Morris <jmorris@redhat.com>
> Err, isn't it "Acked-by:"??  I thought "Signed-off-by:" was only for 
> when
> the patch actually went through someone's tree and was forwarded by them
> to somebody else:

Isn't also a good idea to sign your message with gpg or so? That way one
is 100% sure that it *is* that person who is signing-off or acking-by.


Folkert van Heusden

Op zoek naar een IT of Finance baan? Mail me voor de mogelijkheden!
+------------------------------------------------------------------+
|UNIX admin? Then give MultiTail (http://vanheusden.com/multitail/)|
|a try, it brings monitoring logfiles to a different level! See    |
|http://vanheusden.com/multitail/features.html for a feature list. |
+------------------------------------------= www.unixsoftware.nl =-+
Phone: +31-6-41278122, PGP-key: 1F28D8AE
Get your PGP/GPG key signed at www.biglumber.com!

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 189 bytes --]