r25207: remove one nesting level and use unlikely()

[jelmer/samba4-debian.git] / source / lib / talloc / talloc.c

diff --git a/source/lib/talloc/talloc.c b/source/lib/talloc/talloc.c
index d4c845fa88da1aa46621aa8e72e2a5125e3bbcd9..92970426e80680925ff79dd9b904bdc68734151e 100644 (file)
--- a/source/lib/talloc/talloc.c
+++ b/source/lib/talloc/talloc.c
@@ -1186,18 +1186,18 @@ char *talloc_vasprintf(const void *t, const char *fmt, va_list ap)
        va_copy(ap2, ap);
        len = vsnprintf(&c, 1, fmt, ap2);
        va_end(ap2);
-       if (len < 0) {
+       if (unlikely(len < 0)) {
                return NULL;
        }
 
        ret = (char *)__talloc(t, len+1);
-       if (ret) {
-               va_copy(ap2, ap);
-               vsnprintf(ret, len+1, fmt, ap2);
-               va_end(ap2);
-               _talloc_set_name_const(ret, ret);
-       }
+       if (unlikely(!ret)) return NULL;
+
+       va_copy(ap2, ap);
+       vsnprintf(ret, len+1, fmt, ap2);
+       va_end(ap2);
 
+       _talloc_set_name_const(ret, ret);
        return ret;
 }