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 -- 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