]> git.ipfire.org Git - thirdparty/freeradius-server.git/commitdiff
We actually want a pointer to a NULL pointer because the value box code gets excited
authorArran Cudbard-Bell <a.cudbardb@freeradius.org>
Fri, 26 Jan 2018 02:14:43 +0000 (19:14 -0700)
committerArran Cudbard-Bell <a.cudbardb@freeradius.org>
Fri, 26 Jan 2018 02:14:43 +0000 (19:14 -0700)
src/main/xlat_eval.c

index 1d1bb7a50faf5115cdcecc8927be774ceedee8b8..2fa85a4f94dbe8f18e36cbba43401f143dfb5ee0 100644 (file)
@@ -801,7 +801,7 @@ xlat_action_t xlat_frame_eval(TALLOC_CTX *ctx, fr_cursor_t *out, xlat_exp_t cons
 
                case XLAT_FUNC:
                {
-                       fr_value_box_t empty, *result = &empty;
+                       fr_value_box_t *result = NULL;
 
                        XLAT_DEBUG("** [%i] %s(func) - %%{%s: }", unlang_stack_depth(request), __FUNCTION__,
                                   node->fmt);
@@ -816,8 +816,6 @@ xlat_action_t xlat_frame_eval(TALLOC_CTX *ctx, fr_cursor_t *out, xlat_exp_t cons
                                goto finish;
                        }
 
-                       memset(&empty, 0, sizeof(empty));
-
                        /*
                         *      If there's no children we can just
                         *      call the function directly.