[PULL] Minor bug-fixes discovered by static analysis.

Peter Hutterer peter.hutterer at who-t.net
Sun Mar 7 16:25:59 PST 2010


On Fri, Mar 05, 2010 at 02:12:00PM +0200, Oliver McFadden wrote:
> are available in the git repository at:
> 
>   git://gitorious.org/omcfadde/xserver.git analysis
> 
> Oliver McFadden (5):
>       exa: exaFinishAccess: Overrun of static array "pExaScr->access" of size 6 at position 6 with index variable "i"
>       fb: fbFinishScreenInit: leaked_storage: Variable "(visuals|depths)" goes out of scope
>       parser: xf86readConfigFile: unreachable: This code cannot be reached: "free(val.str);"
>       common: xf86Configure: alloc_strlen: Allocated memory does not have space for the terminating NUL of the string
>       Xext: IdleTimeBlockHandler: unsigned_compare: Comparing unsigned less than zero is never true. "timeout < 0UL"
> 
>  Xext/sync.c                       |    4 +---
>  exa/exa.c                         |    4 ++--
>  fb/fbscreen.c                     |    4 ++++
>  hw/xfree86/common/xf86Configure.c |    2 +-
>  hw/xfree86/parser/read.c          |    4 ++--
>  5 files changed, 10 insertions(+), 8 deletions(-)

fwiw, I find it a lot easier to review patches that land on the mailing list
than a pull request where I have to go off, pull into my tree, review and
then copy/paste segments from the patch to be able to add comments.  it also
makes it harder to track when something has changed, since a tree doesn't
usually contain the "Patch v3" part of it.

Cheers,
  Peter


More information about the xorg-devel mailing list