Coverity: fix missing break in GetOIDText, missing commas in ProcessCrlDistPoints output.

chrome/third_party/mozilla_security_manager/nsNSSCertHelper.cpp

 /* ***** BEGIN LICENSE BLOCK *****
  * Version: MPL 1.1/GPL 2.0/LGPL 2.1
  *
  * The contents of this file are subject to the Mozilla Public License Version
  * 1.1 (the "License"); you may not use this file except in compliance with
  * the License. You may obtain a copy of the License at
  * http://www.mozilla.org/MPL/
  *
  * Software distributed under the License is distributed on an "AS IS" basis,
  * WITHOUT WARRANTY OF ANY KIND, either express or implied. See the License
@@ skipping 350 matching lines @@
       string_id = IDS_CERT_EKU_CODE_SIGNING;
       break;
     case SEC_OID_EXT_KEY_USAGE_EMAIL_PROTECT:
       string_id = IDS_CERT_EKU_EMAIL_PROTECTION;
       break;
     case SEC_OID_EXT_KEY_USAGE_TIME_STAMP:
       string_id = IDS_CERT_EKU_TIME_STAMPING;
       break;
     case SEC_OID_OCSP_RESPONDER:
       string_id = IDS_CERT_EKU_OCSP_SIGNING;
+      break;
     case SEC_OID_PKIX_CPS_POINTER_QUALIFIER:
       string_id = IDS_CERT_PKIX_CPS_POINTER_QUALIFIER;
       break;
     case SEC_OID_PKIX_USER_NOTICE_QUALIFIER:
       string_id = IDS_CERT_PKIX_USER_NOTICE_QUALIFIER;
       break;
 
     // There are a billionty other OIDs we could add here.  I tried to get the
     // important ones...
     default:
@@ skipping 455 matching lines @@
         break;
     }
     if (point->reasons.len) {
       rv += ' ';
       comma = false;
       for (size_t i = 0; i < ARRAYSIZE_UNSAFE(reason_string_map); ++i) {
         if (point->reasons.data[0] & reason_string_map[i].reason) {
           if (comma)
             rv += ',';
           rv += l10n_util::GetStringUTF8(reason_string_map[i].string_id);
+          comma = true;
         }
       }
       rv += '\n';
     }
     if (point->crlIssuer) {
       rv += l10n_util::GetStringFUTF8(
           IDS_CERT_ISSUER_FORMAT,
           UTF8ToUTF16(ProcessGeneralNames(arena.get(), point->crlIssuer)));
     }
   }
@@ skipping 241 matching lines @@
   if (trust.HasPeer(PR_FALSE, PR_TRUE, PR_FALSE) && cert->emailAddr)
     return EMAIL_CERT;
   if (CERT_IsCACert(cert, NULL))
     return CA_CERT;
   if (cert->emailAddr)
     return EMAIL_CERT;
   return UNKNOWN_CERT;
 }
 
 }  // namespace mozilla_security_manager