39 lines
962 B
Diff
39 lines
962 B
Diff
|
From f632c3fd86fce2c495a290dd70f5f09e3e7e7a28 Mon Sep 17 00:00:00 2001
|
||
|
|
From: Hans-Bernhard Broeker <HBBroeker@T-Online.de>
|
||
|
|
Date: Sat, 13 Apr 2019 14:52:35 +0200
|
||
|
|
Subject: [PATCH 5/9] Avoid double-free via double fclose in changestring.
|
||
|
|
|
||
|
|
Mark closed FILE* by setting it to NULL, and check for that.
|
||
|
|
|
||
|
|
Signed-off-by: Vladis Dronov <vdronov@redhat.com>
|
||
|
|
---
|
||
|
|
src/command.c | 5 ++++-
|
||
|
|
1 file changed, 4 insertions(+), 1 deletion(-)
|
||
|
|
|
||
|
|
diff --git a/src/command.c b/src/command.c
|
||
|
|
index 8740b11..dcb5278 100644
|
||
|
|
--- a/src/command.c
|
||
|
|
+++ b/src/command.c
|
||
|
|
@@ -786,6 +786,7 @@ changestring(void)
|
||
|
|
}
|
||
|
|
fprintf(script, "w\nq\n!\n"); /* write and quit */
|
||
|
|
fclose(script);
|
||
|
|
+ script = NULL;
|
||
|
|
|
||
|
|
/* if any line was marked */
|
||
|
|
if (anymarked == YES) {
|
||
|
|
@@ -803,7 +804,9 @@ changestring(void)
|
||
|
|
}
|
||
|
|
changing = NO;
|
||
|
|
mousemenu();
|
||
|
|
- fclose(script);
|
||
|
|
+ if (script != NULL) {
|
||
|
|
+ fclose(script);
|
||
|
|
+ }
|
||
|
|
free(change);
|
||
|
|
return(anymarked);
|
||
|
|
}
|
||
|
|
--
|
||
|
|
2.26.2
|
||
|
|
|