From: Bram Moolenaar Date: Thu, 16 Jun 2022 17:47:20 +0000 (+0100) Subject: patch 8.2.5113: timer becomes invalid after fork/exec, :gui gives errors X-Git-Tag: v8.2.5113 X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=c72e31dfcc013ae840cd7f8899f2430b7f6812c9;p=vim patch 8.2.5113: timer becomes invalid after fork/exec, :gui gives errors Problem: Timer becomes invalid after fork/exec, :gui gives errors. (Gabriel Dupras) Solution: Delete the timer befor forking. (closes #10584) --- diff --git a/src/gui.c b/src/gui.c index 68c64d298..4408545db 100644 --- a/src/gui.c +++ b/src/gui.c @@ -226,6 +226,11 @@ gui_do_fork(void) int exit_status; pid_t pid = -1; +#if defined(FEAT_RELTIME) && defined(HAVE_TIMER_CREATE) + // a timer is not carried forward + delete_timer(); +#endif + // Setup a pipe between the child and the parent, so that the parent // knows when the child has done the setsid() call and is allowed to // exit. diff --git a/src/os_unix.c b/src/os_unix.c index 766deac9a..6fd24508d 100644 --- a/src/os_unix.c +++ b/src/os_unix.c @@ -8247,7 +8247,7 @@ xsmp_close(void) #endif // USE_XSMP #if defined(FEAT_RELTIME) || defined(PROTO) -# if defined(HAVE_TIMER_CREATE) +# if defined(HAVE_TIMER_CREATE) || defined(PROTO) /* * Implement timeout with timer_create() and timer_settime(). */ @@ -8331,6 +8331,19 @@ start_timeout(long msec) return &timeout_flag; } +/* + * To be used before fork/exec: delete any created timer. + */ + void +delete_timer(void) +{ + if (timer_created) + { + timer_delete(timer_id); + timer_created = FALSE; + } +} + # else /* diff --git a/src/proto/os_unix.pro b/src/proto/os_unix.pro index 91024b104..268f3bfb6 100644 --- a/src/proto/os_unix.pro +++ b/src/proto/os_unix.pro @@ -88,4 +88,5 @@ void xsmp_init(void); void xsmp_close(void); void stop_timeout(void); const int *start_timeout(long msec); +void delete_timer(void); /* vim: set ft=c : */ diff --git a/src/version.c b/src/version.c index 66b03c3ab..bfa9c5a26 100644 --- a/src/version.c +++ b/src/version.c @@ -734,6 +734,8 @@ static char *(features[]) = static int included_patches[] = { /* Add new patch number below this line */ +/**/ + 5113, /**/ 5112, /**/