diff --git a/widget/src/os2/nsDeviceContextSpecOS2.cpp b/widget/src/os2/nsDeviceContextSpecOS2.cpp
index 3ec3040a30ef..deaa899b1abf 100644
--- a/widget/src/os2/nsDeviceContextSpecOS2.cpp
+++ b/widget/src/os2/nsDeviceContextSpecOS2.cpp
@@ -698,10 +698,10 @@ int PRINTDLG::GetDefaultPrinter ()
    return mDefaultQueue;
 }
 
-PRINTDLG::GetPrinter (int numPrinter, char** printerName)
+void PRINTDLG::GetPrinter (int numPrinter, char** printerName)
 {
    if (numPrinter > mQueueCount)
-      return NULL;
+      return;
  
    nsCAutoString pName(mPQBuf [numPrinter]->QueueName());
  
@@ -790,7 +790,7 @@ BOOL PRINTDLG::ShowProperties (int index)
     if (buflen != mPQBuf[Ind]->PQI3().pDriverData->cb)
     {
         if (DosAllocMem((PPVOID)&pNewDrivData,buflen,fALLOC))
-            return(DPDM_ERROR);
+            return(FALSE); // DPDM_ERROR
     
 /* copy over old data so driver can use old job */
 /* properties as base for job properties dialog */
diff --git a/widget/src/os2/nsDeviceContextSpecOS2.h b/widget/src/os2/nsDeviceContextSpecOS2.h
index 8d25c4769405..c1802c5143d5 100644
--- a/widget/src/os2/nsDeviceContextSpecOS2.h
+++ b/widget/src/os2/nsDeviceContextSpecOS2.h
@@ -81,7 +81,7 @@ public:
   ~PRINTDLG ();
    int GetNumPrinters ();
    int GetDefaultPrinter ();
-   GetPrinter (int numPrinter, char** printerName);
+   void GetPrinter (int numPrinter, char** printerName);
    PRTQUEUE* SetPrinterQueue (int numPrinter);
    HDC GetDCHandle (int numPrinter);
    PLONG GetPrintDriverSize (int printer);