On Sat, Jul 31, 2010 at 21:09 +0200, Dan Carpenter wrote: > On Sat, Jul 31, 2010 at 09:38:00PM +0400, Kulikov Vasiliy wrote: > > - sizeof(struct m68k_serial))) > > - return -EFAULT; > > - copy_to_user((struct m68k_serial *) arg, > > + return copy_to_user((struct m68k_serial *) arg, > > info, sizeof(struct m68k_serial)); > > We should return if -EFAULT copy_to_user() failes here. Right, I was looking for this bug and made it myself :) Thank you. > > > - return 0; > > - > > default: > > return -ENOIOCTLCMD; > > } > > Smatch would have caught that but I don't have a cross compile > environment set up. > > regards, > dan carpenter -- To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
