From e27175c7c8cdfeb4e28d4ff21eb51aa003d70a03 Mon Sep 17 00:00:00 2001 From: Kamil Dudka Date: Wed, 7 Nov 2018 14:04:55 +0100 Subject: [PATCH] 43789: possible use after free clearing up math func from module --- ChangeLog | 3 +++ Src/module.c | 2 -- 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/ChangeLog b/ChangeLog index 009c33e30..343038997 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,5 +1,8 @@ 2018-11-09 Peter Stephenson + * 43789: Kamil: Src/module.c: possible use after free handling + math functions from module. + * 43792: Kamil: Src/exec.c: file descriptor could leak on fork error. * 43793: Kamil: Src/Zle/computil.c: could overrun buffer by 1 byte. diff --git a/Src/module.c b/Src/module.c index 4ae78310f..33d75ebbd 100644 --- a/Src/module.c +++ b/Src/module.c @@ -1390,8 +1390,6 @@ setmathfuncs(char const *nam, MathFunc f, int size, int *e) if (deletemathfunc(f)) { zwarnnam(nam, "math function `%s' already deleted", f->name); ret = 1; - } else { - f->flags &= ~MFF_ADDED; } } f++;