[PATCH] staging: ks7010: fix off by one error

Tobin C. Harding me at tobin.cc
Wed Mar 15 02:47:03 UTC 2017


On Wed, Mar 15, 2017 at 09:48:55AM +0800, Greg Kroah-Hartman wrote:
> On Wed, Mar 15, 2017 at 09:48:14AM +0800, Greg Kroah-Hartman wrote:
> > On Wed, Mar 15, 2017 at 12:01:07PM +1100, Tobin C. Harding wrote:
> > > Bug introduced in commit 7676b72 by Tobin C. Harding.
> > > 
> > > Remove equals sign from comparison, fixing off by one error.
> > > 
> > > Signed-off-by: Tobin C. Harding <me at tobin.cc>
> > 
> > In the future, when you fix something, it's nice to add a tag saying so
> > in this format:
> > Fixes: 7676b72428e8 ("staging: ks7010: move comparison to right hand side")
> > 
> > You can generate the string by doing:
> > 	git show -s --abbrev-commit --abbrev=12 --pretty=format:"%h (\"%s\")%n" 7676b72
> > 
> > I'll add it for this one.
> 
> Also, it's good to give credit to the person who found the bug:
> 	Reported-by: Dan Carpenter <dan.carpenter at oracle.com>

Thank you, this is clearly stated in
Documentation/process/submitting-patches.rst since the bug was
reported on a mailing list.  My mistake.

Got it now.

thanks,
Tobin.


More information about the devel mailing list