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

Marcus Meissner marcus at jet.franken.de
Sun Feb 1 09:07:01 CST 2009


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, &needed,
> > &num) && needed) {
> 
> EnumPrinterA must always update "needed".
> "nedded" is 0, when EnumPrintersA failed or when no printer is
> installed.
> 
> Line 1555 has similar code

Hmm, yes. I see the initialization in WINSPOOL_EnumPrinters() now. :)

Ciao, Marcus



More information about the wine-devel mailing list