Bug 1229881 - fix off-by-one error in nsPrinterEnumeratorWin::GetPrinterNameList; r=dbaron; a=KWierso

This commit is contained in:
Nathan Froyd 2015-12-02 14:28:38 -05:00
parent 0f9a85616c
commit b3b42fa2a9

View File

@ -687,10 +687,9 @@ nsPrinterEnumeratorWin::GetPrinterNameList(nsIStringEnumerator **aPrinterNameLis
if (!printers)
return NS_ERROR_OUT_OF_MEMORY;
uint32_t printerInx = 0;
nsString* names = printers->AppendElements(numPrinters);
while( printerInx < numPrinters ) {
LPWSTR name = GlobalPrinters::GetInstance()->GetItemFromList(printerInx++);
for (uint32_t printerInx = 0; printerInx < numPrinters; ++printerInx) {
LPWSTR name = GlobalPrinters::GetInstance()->GetItemFromList(printerInx);
names[printerInx].Assign(name);
}