[flashrom] [PATCH] Fix a number of problems in mstarddc_spi.c.
Stefan Tauner
stefan.tauner at alumni.tuwien.ac.at
Thu Feb 19 00:29:26 CET 2015
On Wed, 18 Feb 2015 23:50:49 +0100
Alexandre Boeglin <alex at boeglin.org> wrote:
> Le mercredi 18 février 2015 à 01:31, Stefan Tauner a écrit:
> > Coverity has brought up the following problems:
>
> Sorry, I wasn't aware that there were coverity reports for the project.
the coverity project is invite-only anyway... and there is not much to
see usually because of the rather clean code base :)
https://scan.coverity.com/projects/1020?tab=overview
> > This patch revamps the function in various ways to fix these issues and some
> > other irritating bits.
> > It reduces scopes of variables where possible, pushes the code towards our
> > coding standards and introduces a label-based resource cleanup at the end.
>
> I agree, this patch does make the code easier to read, in addition to
> fixing my mistakes :)
Thanks, I'll take that as an acked-by... because the code is disabled by
default anyway ;)
Committed in r1885.
--
Kind regards/Mit freundlichen Grüßen, Stefan Tauner
More information about the flashrom
mailing list