On Fri, 2013-09-13 at 15:29 +0300, Dan Carpenter wrote: > If new_mtu is very large then "new_mtu + ETH_HLEN + ETH_FCS_LEN" can > wrap and the check on the next line can underflow. This is one of > those > bugs which can be triggered by the user if you have namespaces > configured. > > Also since this is something the user can trigger then we don't want > to > have dev_err() message. > > This is a static checker fix and I'm not sure what the impact is. > --- > v2: reformat and also remove the dev_err() > > Signed-off-by: Dan Carpenter <[email protected]>
Thanks Dan, I have added your patch to my queue.
signature.asc
Description: This is a digitally signed message part
------------------------------------------------------------------------------ How ServiceNow helps IT people transform IT departments: 1. Consolidate legacy IT systems to a single system of record for IT 2. Standardize and globalize service processes across IT 3. Implement zero-touch automation to replace manual, redundant tasks http://pubads.g.doubleclick.net/gampad/clk?id=51271111&iu=/4140/ostg.clktrk
_______________________________________________ E1000-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/e1000-devel To learn more about Intel® Ethernet, visit http://communities.intel.com/community/wired
