files: Only copy pairs if there are pairs to copy (ubsan)

FreeRADIUS · Jun 4, 2020 · 0f32839 · 0f32839
1 parent 21a5208
commit 0f32839
Showing 1 changed file with 4 additions and 3 deletions.
diff --git a/src/modules/rlm_files/rlm_files.c b/src/modules/rlm_files/rlm_files.c
@@ -359,9 +359,10 @@ static rlm_rcode_t file_common(rlm_files_t const *inst, REQUEST *request, char c
 			found = true;
 
 			/* ctx may be reply */
-			MEM(fr_pair_list_copy(reply, &reply_tmp, pl->reply) >= 0);
-
-			radius_pairmove(request, &reply->vps, reply_tmp, true);
+			if (pl->reply) {
+				MEM(fr_pair_list_copy(reply, &reply_tmp, pl->reply) >= 0);
+				radius_pairmove(request, &reply->vps, reply_tmp, true);
+			}
 			fr_pair_list_move(&request->control, &check_tmp);
 
 			reply_tmp = NULL;	/* radius_pairmove() frees input attributes */