From: James Jones <jejones3141@gmail.com>
Date: Thu, 19 May 2022 22:59:43 +0000 (-0500)
Subject: Check return values in rlm_date.c (CID #1503947, #1503971) (#4520)
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=f1972f8907488961df9e7b60f15baf9e91a7a309;p=thirdparty%2Ffreeradius-server.git

Check return values in rlm_date.c (CID #1503947, #1503971) (#4520)
---

diff --git a/src/modules/rlm_date/rlm_date.c b/src/modules/rlm_date/rlm_date.c
index 4186b4ad2d4..ddebcdc4df9 100644
--- a/src/modules/rlm_date/rlm_date.c
+++ b/src/modules/rlm_date/rlm_date.c
@@ -100,7 +100,7 @@ static xlat_action_t date_convert_string(TALLOC_CTX *ctx, fr_dcursor_t *out, req
 		return XLAT_ACTION_FAIL;
 	}
 
-	vb = fr_value_box_alloc(ctx, FR_TYPE_DATE, NULL, false);
+	MEM(vb = fr_value_box_alloc(ctx, FR_TYPE_DATE, NULL, false));
 	vb->vb_date = fr_unix_time_from_sec(date);
 	fr_dcursor_append(out, vb);
 	return XLAT_ACTION_DONE;
@@ -127,8 +127,8 @@ static xlat_action_t date_encode_strftime(TALLOC_CTX *ctx, fr_dcursor_t *out, rl
 
 	if (strftime(buff, sizeof(buff), inst->fmt, &tminfo) == 0) return XLAT_ACTION_FAIL;
 
-	vb = fr_value_box_alloc_null(ctx);
-	fr_value_box_strdup(ctx, vb, NULL, buff, false);
+	MEM(vb = fr_value_box_alloc_null(ctx));
+	MEM(fr_value_box_strdup(ctx, vb, NULL, buff, false) == 0);
 	fr_dcursor_append(out, vb);
 
 	return XLAT_ACTION_DONE;