Bug 1684088 - Check return value of NS_MsgGetOperatorFromString() before using the return argument to avoid using uninitialized value. r=mkmelin

mailnews/search/src/nsMsgSearchTerm.cpp

@@ -574,9 +574,9 @@ nsresult nsMsgSearchTerm::ParseOperator(char* inStream,
 
   if (commaSep) *commaSep = '\0';
 
-  nsresult err = NS_MsgGetOperatorFromString(inStream, &operatorVal);
-  *value = (nsMsgSearchOpValue)operatorVal;
-  return err;
+  nsresult rv = NS_MsgGetOperatorFromString(inStream, &operatorVal);
+  if (NS_SUCCEEDED(rv)) *value = (nsMsgSearchOpValue)operatorVal;
+  return rv;
 }
 
 // find the attribute code for this comma-delimited attribute.