Skip to content

Commit a268d9e

Browse files
authored
Merge pull request #12757 from Alizter/push-loplvmpotntt
fix(fswatch_win): avoid use after free()
2 parents b0b740b + 7aba4da commit a268d9e

File tree

1 file changed

+4
-1
lines changed

1 file changed

+4
-1
lines changed

src/fswatch_win/fswatch_win_stubs.c

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -502,10 +502,13 @@ value fswatch_win_shutdown(value v_fsenv) {
502502
CloseHandle(fsenv->signal);
503503
CloseHandle(fsenv->afterAdd);
504504

505-
for (struct events *e = pop_events(fsenv); e; e = e->next) {
505+
struct events *e = pop_events(fsenv);
506+
while (e != NULL) {
506507
free(e->path);
507508
free(e->buffer);
509+
struct events *next = e->next;
508510
free(e);
511+
e = next;
509512
}
510513

511514
caml_acquire_runtime_system();

0 commit comments

Comments
 (0)