On Mon, 29 Oct 2012 12:10:29 -0500 Shirish Pargaonkar <shirishpargaonkar@xxxxxxxxx> wrote: > On Sat, Oct 27, 2012 at 6:20 AM, Jeff Layton <jlayton@xxxxxxxxx> wrote: > > This patchset does a number of cleanups to the cifs.idmap request-key > > callout program. It also makes it so that it will set a timeout on the > > keys that it instantiates. > > > > Most of the fixes are to add extra bounds checking, fix unsigned vs. > > signed type confusion, and to avoid unnecessary memory allocations. > > > > Jeff Layton (7): > > cifs.idmap: make sure cifsacl structs are packed > > cifs.idmap: get rid of useless strcmp prior to idmapping > > cifs.idmap: add an options struct to handle long options > > cifs.idmap: clean up strget and avoid memory allocation > > cifs.idmap: don't use atoi to convert unsigned int to number > > cifs.idmap: set a timeout on keys that it instantiates > > cifs.idmap: add a --help option for cifs.idmap > > > > cifs.idmap.8.in | 12 +++++- > > cifs.idmap.c | 113 +++++++++++++++++++++++++++++++++++++++++--------------- > > cifsacl.h | 8 ++-- > > 3 files changed, 98 insertions(+), 35 deletions(-) > > > > -- > > 1.7.11.7 > > > > Looks correct. > > Reviewed-by: Shirish Pargaonkar <shirishpargaonkar@xxxxxxxxx> Thanks Shirish. I'll merge them soon. -- Jeff Layton <jlayton@xxxxxxxxx> -- To unsubscribe from this list: send the line "unsubscribe linux-cifs" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html