> On Aug 7, 2015, at 1:51 PM, Jan Kara <jack@xxxxxxxx> wrote: > > Signed-off-by: Jan Kara <jack@xxxxxxxx> > --- > > + new_first_ino = strtoul(arg, &p, 0); > + if (*p) { > + fprintf(stderr, > + _("Invalid number of reserved inodes " > + "%s\n"), > + arg); > + r_usage++; > + continue; > + } > this check invalid - you need check an errno for ERANGE also. add check against maximal number inodes on FS is also good. -- To unsubscribe from this list: send the line "unsubscribe linux-ext4" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html