]> git.ipfire.org Git - thirdparty/samba.git/commitdiff
gse: Simplify gse_errstr() with talloc_asprintf_addbuf()
authorVolker Lendecke <vl@samba.org>
Wed, 29 May 2024 15:27:11 +0000 (17:27 +0200)
committerJeremy Allison <jra@samba.org>
Mon, 24 Jun 2024 16:21:32 +0000 (16:21 +0000)
Signed-off-by: Volker Lendecke <vl@samba.org>
Reviewed-by: Jeremy Allison <jra@samba.org>
source3/librpc/crypto/gse.c

index 356356dfcac6e053fa8ad97f54db4517c4c9f7f2..a071025c6f64244a16b1e1500e1c7f81674bd486 100644 (file)
@@ -834,9 +834,7 @@ static char *gse_errstr(TALLOC_CTX *mem_ctx, OM_uint32 maj, OM_uint32 min)
        errstr = talloc_strndup(mem_ctx,
                                (char *)msg_maj.value,
                                        msg_maj.length);
-       if (!errstr) {
-               goto done;
-       }
+
        gss_maj = gss_display_status(&gss_min, min, GSS_C_MECH_CODE,
                                     (gss_OID)discard_const(gss_mech_krb5),
                                     &msg_ctx, &msg_min);
@@ -844,16 +842,10 @@ static char *gse_errstr(TALLOC_CTX *mem_ctx, OM_uint32 maj, OM_uint32 min)
                goto done;
        }
 
-       errstr = talloc_strdup_append_buffer(errstr, ": ");
-       if (!errstr) {
-               goto done;
-       }
-       errstr = talloc_strndup_append_buffer(errstr,
-                                               (char *)msg_min.value,
-                                                       msg_min.length);
-       if (!errstr) {
-               goto done;
-       }
+       talloc_asprintf_addbuf(&errstr,
+                              ": %.*s",
+                              (int)msg_min.length,
+                              (char *)msg_min.value);
 
 done:
        if (msg_min.value) {