Mercurial > vim
diff src/channel.c @ 9058:87c2e43a4a12 v7.4.1814
commit https://github.com/vim/vim/commit/b8d4905592fc26fcd09180d7d6bfefd899f2f6c6
Author: Bram Moolenaar <Bram@vim.org>
Date: Sun May 1 14:22:16 2016 +0200
patch 7.4.1814
Problem: A channel may be garbage collected while it's still being used by
a job. (James McCoy)
Solution: Mark the channel as used if the job is still used. Do the same
for channels that are still used.
| author | Christian Brabandt <cb@256bit.org> |
|---|---|
| date | Sun, 01 May 2016 14:30:06 +0200 |
| parents | 31bba6f25d84 |
| children | a86103d4b356 |
line wrap: on
line diff
--- a/src/channel.c +++ b/src/channel.c @@ -3553,28 +3553,15 @@ set_ref_in_channel(int copyID) { int abort = FALSE; channel_T *channel; - int part; + typval_T tv; for (channel = first_channel; channel != NULL; channel = channel->ch_next) - { - for (part = PART_SOCK; part < PART_IN; ++part) + if (channel_still_useful(channel)) { - jsonq_T *head = &channel->ch_part[part].ch_json_head; - jsonq_T *item = head->jq_next; - - while (item != NULL) - { - list_T *l = item->jq_value->vval.v_list; - - if (l->lv_copyID != copyID) - { - l->lv_copyID = copyID; - abort = abort || set_ref_in_list(l, copyID, NULL); - } - item = item->jq_next; - } + tv.v_type = VAR_CHANNEL; + tv.vval.v_channel = channel; + abort = abort || set_ref_in_item(&tv, copyID, NULL, NULL); } - } return abort; } @@ -4092,6 +4079,26 @@ job_still_useful(job_T *job) && channel_still_useful(job->jv_channel))); } +/* + * Mark references in jobs that are still useful. + */ + int +set_ref_in_job(int copyID) +{ + int abort = FALSE; + job_T *job; + typval_T tv; + + for (job = first_job; job != NULL; job = job->jv_next) + if (job_still_useful(job)) + { + tv.v_type = VAR_JOB; + tv.vval.v_job = job; + abort = abort || set_ref_in_item(&tv, copyID, NULL, NULL); + } + return abort; +} + void job_unref(job_T *job) {