librpc:ndr: Move call to convert_string_talloc() on to its own line
authorJoseph Sutton <josephsutton@catalyst.net.nz>
Thu, 16 Nov 2023 01:50:58 +0000 (14:50 +1300)
committerAndrew Bartlett <abartlet@samba.org>
Thu, 16 Nov 2023 05:18:36 +0000 (05:18 +0000)
Signed-off-by: Joseph Sutton <josephsutton@catalyst.net.nz>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
librpc/ndr/ndr_string.c

index 983e248c0f1f09131b918ee0d102964762e0b393..5e8eb895bf7332f051b3a9bb9dd9843df509a997 100644 (file)
@@ -319,11 +319,15 @@ _PUBLIC_ enum ndr_err_code ndr_push_string(struct ndr_push *ndr, ndr_flags_type
                        return ndr_push_error(ndr, NDR_ERR_ALLOC,
                                              "Failed to talloc_strndup() in ndr_string_push()");
                }
-       } else if (!convert_string_talloc(ndr, CH_UNIX, chset, s, s_len,
-                                         &dest, &d_len))
-       {
-               return ndr_push_error(ndr, NDR_ERR_CHARCNV,
-                                     "Bad character push conversion with flags 0x%"PRI_LIBNDR_FLAGS, flags);
+       } else {
+               bool ok;
+
+               ok = convert_string_talloc(ndr, CH_UNIX, chset, s, s_len,
+                                          &dest, &d_len);
+               if (!ok) {
+                       return ndr_push_error(ndr, NDR_ERR_CHARCNV,
+                                             "Bad character push conversion with flags 0x%"PRI_LIBNDR_FLAGS, flags);
+               }
        }
 
        if (flags & LIBNDR_FLAG_STR_BYTESIZE) {