Bug 443284, crash on login to ActivIdentity/ActivClient smartcard using acpkcs211.dll

Patches contributed by timeless and Dani Church
r=kaie, r=rrelyea

security/manager/ssl/src/nsNSSCallbacks.cpp

@@ -724,10 +724,10 @@ ShowProtectedAuthPrompt(PK11SlotInfo* slot, nsIInterfaceRequestor *ir)
           switch (rv)
           {
               case SECSuccess:
-                  protAuthRetVal = PK11_PW_AUTHENTICATED;
+                  protAuthRetVal = ToNewCString(nsDependentCString(PK11_PW_AUTHENTICATED));
                   break;
               case SECWouldBlock:
-                  protAuthRetVal = PK11_PW_RETRY;
+                  protAuthRetVal = ToNewCString(nsDependentCString(PK11_PW_RETRY));
                   break;
               default:
                   protAuthRetVal = nsnull;

security/manager/ssl/src/nsProtectedAuthThread.cpp

@@ -158,7 +158,7 @@ void nsProtectedAuthThread::Run(void)
     // it is harmless here
     mLoginResult = PK11_CheckUserPassword(mSlot, 0);
 
-    nsIObserver *observer = nsnull;
+    nsCOMPtr<nsIObserver> observer;
     
     PR_Lock(mMutex);