Re: [PATCH] winspool.drv: Check return value of EnumPrintersA

2009-02-01 Thread Marcus Meissner
On Sun, Feb 01, 2009 at 03:25:43PM +0100, Detlef Riekenberg wrote: > On Sa, 2009-01-31 at 11:09 +0100, Marcus Meissner wrote: > > -EnumPrintersA(PRINTER_ENUM_LOCAL, NULL, 5, NULL, 0, &needed, > > &num); > > -if(needed) { > > +if (EnumPrintersA(PRINTER_ENUM_LOCAL, NULL, 5, NULL, 0, &need

Re: [PATCH] winspool.drv: Check return value of EnumPrintersA

2009-02-01 Thread Detlef Riekenberg
On Sa, 2009-01-31 at 11:09 +0100, Marcus Meissner wrote: > -EnumPrintersA(PRINTER_ENUM_LOCAL, NULL, 5, NULL, 0, &needed, > &num); > -if(needed) { > +if (EnumPrintersA(PRINTER_ENUM_LOCAL, NULL, 5, NULL, 0, &needed, > &num) && needed) { EnumPrinterA must always update "needed". "nedded"