[prev in list] [next in list] [prev in thread] [next in thread] 

List:       apr-cvs
Subject:    svn commit: r1875220 - /apr/apr/trunk/jose/apr_jose.c
From:       minfrin () apache ! org
Date:       2020-03-16 0:34:02
Message-ID: 20200316003402.5088517AE52 () svn01-us-east ! apache ! org
[Download RAW message or body]

Author: minfrin
Date: Mon Mar 16 00:34:02 2020
New Revision: 1875220

URL: http://svn.apache.org/viewvc?rev=1875220&view=rev
Log:
apr_jose: Fix a the case where existing memory buffers were used, but they
were not initialised.

Modified:
    apr/apr/trunk/jose/apr_jose.c

Modified: apr/apr/trunk/jose/apr_jose.c
URL: http://svn.apache.org/viewvc/apr/apr/trunk/jose/apr_jose.c?rev=1875220&r1=1875219&r2=1875220&view=diff
 ==============================================================================
--- apr/apr/trunk/jose/apr_jose.c (original)
+++ apr/apr/trunk/jose/apr_jose.c Mon Mar 16 00:34:02 2020
@@ -42,11 +42,9 @@ APR_DECLARE(apr_jose_t *) apr_jose_data_
         const unsigned char *in, apr_size_t inlen, apr_pool_t *pool)
 {
 
+    jose = apr_jose_make(jose, APR_JOSE_TYPE_DATA, pool);
     if (!jose) {
-        jose = apr_jose_make(jose, APR_JOSE_TYPE_DATA, pool);
-        if (!jose) {
-            return NULL;
-        }
+        return NULL;
     }
 
     jose->typ = typ;
@@ -64,11 +62,9 @@ APR_DECLARE(apr_jose_t *) apr_jose_json_
         apr_json_value_t *json, apr_pool_t *pool)
 {
 
+    jose = apr_jose_make(jose, APR_JOSE_TYPE_JSON, pool);
     if (!jose) {
-        jose = apr_jose_make(jose, APR_JOSE_TYPE_JSON, pool);
-        if (!jose) {
-            return NULL;
-        }
+        return NULL;
     }
 
     jose->cty = cty;
@@ -141,11 +137,9 @@ APR_DECLARE(apr_jose_t *) apr_jose_jwe_m
 {
     apr_jose_jwe_t *jwe;
 
+    jose = apr_jose_make(jose, APR_JOSE_TYPE_JWE, pool);
     if (!jose) {
-        jose = apr_jose_make(jose, APR_JOSE_TYPE_JWE, pool);
-        if (!jose) {
-            return NULL;
-        }
+        return NULL;
     }
 
     jose->cty = payload->cty;
@@ -170,11 +164,9 @@ APR_DECLARE(apr_jose_t *) apr_jose_jwe_j
 {
     apr_jose_jwe_t *jwe;
 
+    jose = apr_jose_make(jose, APR_JOSE_TYPE_JWE_JSON, pool);
     if (!jose) {
-        jose = apr_jose_make(jose, APR_JOSE_TYPE_JWE_JSON, pool);
-        if (!jose) {
-            return NULL;
-        }
+        return NULL;
     }
 
     if (payload) {
@@ -199,11 +191,9 @@ APR_DECLARE(apr_jose_t *) apr_jose_jwk_m
 {
     apr_jose_jwk_t *jwk;
 
+    jose = apr_jose_make(jose, APR_JOSE_TYPE_JWK, pool);
     if (!jose) {
-        jose = apr_jose_make(jose, APR_JOSE_TYPE_JWK, pool);
-        if (!jose) {
-            return NULL;
-        }
+        return NULL;
     }
 
     jwk = jose->jose.jwk = apr_palloc(pool, sizeof(apr_jose_jwk_t));
@@ -221,11 +211,9 @@ APR_DECLARE(apr_jose_t *) apr_jose_jwks_
 {
     apr_jose_jwks_t *jwks;
 
+    jose = apr_jose_make(jose, APR_JOSE_TYPE_JWKS, pool);
     if (!jose) {
-        jose = apr_jose_make(jose, APR_JOSE_TYPE_JWKS, pool);
-        if (!jose) {
-            return NULL;
-        }
+        return NULL;
     }
 
     jwks = jose->jose.jwks = apr_palloc(pool, sizeof(apr_jose_jwks_t));
@@ -244,11 +232,9 @@ APR_DECLARE(apr_jose_t *) apr_jose_jws_m
 {
     apr_jose_jws_t *jws;
 
+    jose = apr_jose_make(jose, APR_JOSE_TYPE_JWS, pool);
     if (!jose) {
-        jose = apr_jose_make(jose, APR_JOSE_TYPE_JWS, pool);
-        if (!jose) {
-            return NULL;
-        }
+        return NULL;
     }
 
     if (payload) {
@@ -273,11 +259,9 @@ APR_DECLARE(apr_jose_t *) apr_jose_jws_j
 {
     apr_jose_jws_t *jws;
 
+    jose = apr_jose_make(jose, APR_JOSE_TYPE_JWS_JSON, pool);
     if (!jose) {
-        jose = apr_jose_make(jose, APR_JOSE_TYPE_JWS_JSON, pool);
-        if (!jose) {
-            return NULL;
-        }
+        return NULL;
     }
 
     if (payload) {
@@ -301,11 +285,9 @@ APR_DECLARE(apr_jose_t *) apr_jose_jwt_m
 {
     apr_jose_jwt_t *jwt;
 
+    jose = apr_jose_make(jose, APR_JOSE_TYPE_JWT, pool);
     if (!jose) {
-        jose = apr_jose_make(jose, APR_JOSE_TYPE_JWT, pool);
-        if (!jose) {
-            return NULL;
-        }
+        return NULL;
     }
 
     jose->cty = "JWT";
@@ -324,11 +306,9 @@ APR_DECLARE(apr_jose_t *) apr_jose_text_
         const char *in, apr_size_t inlen, apr_pool_t *pool)
 {
 
+    jose = apr_jose_make(jose, APR_JOSE_TYPE_TEXT, pool);
     if (!jose) {
-        jose = apr_jose_make(jose, APR_JOSE_TYPE_TEXT, pool);
-        if (!jose) {
-            return NULL;
-        }
+        return NULL;
     }
 
     jose->cty = cty;


[prev in list] [next in list] [prev in thread] [next in thread]