Messages in this thread | | | From | Kyle Moffett <> | Subject | Re: [PATCH] SELinux: null dereference in error path | Date | Mon, 28 Feb 2005 23:25:27 -0500 |
| |
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 <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:
EG: John Doe writes a patch that fixes a NULL pointer deref, and he sends it to Andrew Morton. The maintainer of the driver, Jane McDonald, confirms the fix via email to this list. Andrew forwards it to Linus, who includes it in his next release. The resulting notations look like this:
Signed-off-by: John Doe Acked-by: Jane McDonald Signed-off-by: Andrew Morton Signed-off-by: Linus Torvalds
Cheers, Kyle Moffett
-----BEGIN GEEK CODE BLOCK----- Version: 3.12 GCM/CS/IT/U d- s++: a18 C++++>$ UB/L/X/*++++(+)>$ P+++(++++)>$ L++++(+++) E W++(+) N+++(++) o? K? w--- O? M++ V? PS+() PE+(-) Y+ PGP+++ t+(+++) 5 X R? tv-(--) b++++(++) DI+ D+ G e->++++$ h!*()>++$ r !y?(-) ------END GEEK CODE BLOCK------
- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |