Commit 1a568d08 authored by wtchang%redhat.com's avatar wtchang%redhat.com
Browse files

Bugzilla bug 240554: set (better) error codes and removed an unreachable 

break statement. r=relyea.
parent c0bd0e74

security/nss/lib/cryptohi/secsign.c

+3 −3
Original line number Diff line number Diff line
@@ -37,7 +37,7 @@ 
 * the terms of any one of the MPL, the GPL or the LGPL.

 *

 * ***** END LICENSE BLOCK ***** */

/* $Id: secsign.c,v 1.10 2005/08/11 23:11:37 wtchang%redhat.com Exp $ */

/* $Id: secsign.c,v 1.11 2005/08/12 23:24:22 wtchang%redhat.com Exp $ */



#include <stdio.h>

#include "cryptohi.h"
@@ -132,7 +132,7 @@ SGN_NewContext(SECOidTag alg, SECKEYPrivateKey *key) 
       * we *CERTAINLY* don't want to sign one! */

      case SEC_OID_PKCS1_MD4_WITH_RSA_ENCRYPTION:

      default:

	PORT_SetError(SEC_ERROR_INVALID_ARGS);

	PORT_SetError(SEC_ERROR_INVALID_ALGORITHM);

	return 0;

    }
@@ -431,8 +431,8 @@ SEC_DerSignData(PRArenaPool *arena, SECItem *result, 
	    break;

#endif /* NSS_ENABLE_ECC */

	  default:

	    PORT_SetError(SEC_ERROR_INVALID_KEY);

	    return SECFailure;

	    break;

	}

    }