On Tue, May 10, 2022 at 5:02 PM Christian Göttsche <cgzones@xxxxxxxxxxxxxx> wrote: > > In case the allocation for the filename fails, free the memory of the context. > > Signed-off-by: Christian Göttsche <cgzones@xxxxxxxxxxxxxx> Acked-by: James Carter <jwcart2@xxxxxxxxx> > --- > libselinux/src/selinux_restorecon.c | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) > > diff --git a/libselinux/src/selinux_restorecon.c b/libselinux/src/selinux_restorecon.c > index e6192912..e160c06e 100644 > --- a/libselinux/src/selinux_restorecon.c > +++ b/libselinux/src/selinux_restorecon.c > @@ -489,13 +489,15 @@ static int filespec_add(ino_t ino, const char *con, const char *file, > goto oom_freefl; > fl->file = strdup(file); > if (!fl->file) > - goto oom_freefl; > + goto oom_freeflcon; > fl->next = prevfl->next; > prevfl->next = fl; > > __pthread_mutex_unlock(&fl_mutex); > return 0; > > +oom_freeflcon: > + free(fl->con); > oom_freefl: > free(fl); > oom: > -- > 2.36.1 >
