You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@couchdb.apache.org by ja...@apache.org on 2019/02/23 03:57:49 UTC

[couchdb-b64url] 08/31: Attempting to avoid a compiler warning on Jenkins

This is an automated email from the ASF dual-hosted git repository.

jaydoane pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/couchdb-b64url.git

commit edaf27bc96c9384558df4742e77a8bf85d989eda
Author: Paul J. Davis <pa...@gmail.com>
AuthorDate: Thu Nov 21 15:29:13 2013 -0600

    Attempting to avoid a compiler warning on Jenkins
---
 c_src/b64url.c | 14 +++++++++-----
 1 file changed, 9 insertions(+), 5 deletions(-)

diff --git a/c_src/b64url.c b/c_src/b64url.c
index 0bc6ab8..3528d3b 100644
--- a/c_src/b64url.c
+++ b/c_src/b64url.c
@@ -546,28 +546,32 @@ b64url_decode_cont(ErlNifEnv* env, int argc, const ENTERM argv[])
     int status;
 
     if(argc != 2) {
-        return enif_make_badarg(env);
+        goto badarg;
     }
 
     if(!enif_inspect_iolist_as_binary(env, argv[0], &src)) {
-        return enif_make_badarg(env);
+        goto badarg;
     }
 
     if(!enif_get_resource(env, argv[1], priv->res_st, (void**) &st)) {
-        return enif_make_badarg(env);
+        goto badarg;
     }
 
     if(!check_pid(env, st)) {
-        return enif_make_badarg(env);
+        goto badarg;
     }
 
     if(src.size != st->len) {
-        return enif_make_badarg(env);
+        goto badarg;
     }
 
     status = b64url_decode(env, &src, st, &ret);
 
     return b64url_st_dec_ret(env, st, status, ret);
+
+badarg:
+    ret = enif_make_badarg(env);
+    return ret;
 }