From 5a60ea6ee0b04500c53031bc004097809c1e7b7d Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Deuc=D0=B5?= <shurd@sasktel.net>
Date: Tue, 16 Feb 2021 12:26:58 -0500
Subject: [PATCH] Only patch I looked at in 68990cd was wrong.
x points to the terminating NUL in x64. free()ing it would do wild
and crazy things depending on how paranoid the free() implementation
is.
free(x64) instead... at least Coverity was happy though!
---
src/sbbs3/js_cryptcon.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/sbbs3/js_cryptcon.c b/src/sbbs3/js_cryptcon.c
index e4f319e76d..19fd936fdf 100644
--- a/src/sbbs3/js_cryptcon.c
+++ b/src/sbbs3/js_cryptcon.c
@@ -131,7 +131,7 @@ static int js_ecc_to_prop(unsigned char *data, size_t len, size_t *off, JSContex
memcpy(y, z, zcnt);
y64 = malloc(zcnt*4/3+3);
if (y64 == NULL) {
- free(x);
+ free(x64);
free(y);
return 0;
}
--
GitLab