thac0/vim
1
0
Fork 0
forked from aniani/vim
Code Activity

patch 7.4.1824

Browse Source 
Problem:    When a job is no longer referenced and does not have an exit
            callback the process may hang around in defunc state. (Nicola)
Solution:   Call job_status() if the job is running and won't get freed
            because it might still be useful.
This commit is contained in:
Bram Moolenaar
2016-05-08 13:21:12 +02:00
parent d9db8b448c
commit 36e0f7da9b
2 changed files with 5 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
src/channel.c
View File

@@ -4144,7 +4144,7 @@ free_unused_jobs_contents(int copyID, int mask)
* recurse into Lists, Dictionaries etc. */ * recurse into Lists, Dictionaries etc. */
job_free_contents(job); job_free_contents(job);
did_free = TRUE; did_free = TRUE;
} }
return did_free; return did_free;
} }
@@ -4234,7 +4234,7 @@ job_stop_on_exit()
} }
/* /*
* Called once in a while: check if any jobs with an "exit_cb" have ended. * Called once in a while: check if any jobs that seem useful have ended.
*/ */
void void
job_check_ended(void) job_check_ended(void)
@@ -4252,7 +4252,7 @@ job_check_ended(void)
for (job = first_job; job != NULL; job = next) for (job = first_job; job != NULL; job = next)
{ {
next = job->jv_next; next = job->jv_next;
if (job->jv_status == JOB_STARTED && job->jv_exit_cb != NULL) if (job->jv_status == JOB_STARTED && job_still_useful(job))
job_status(job); /* may free "job" */ job_status(job); /* may free "job" */
} }
} }

2
src/version.c
View File

@@ -753,6 +753,8 @@ static char *(features[]) =
static int included_patches[] = static int included_patches[] =
{ /* Add new patch number below this line */ { /* Add new patch number below this line */
/**/
1824,
/**/ /**/
1823, 1823,
/**/ /**/