diff options
| author |   nicm <nicm@openbsd.org> | 2009-10-11 07:30:07 +0000 |
|---|---|---|
| committer | nicm <nicm@openbsd.org> | 2009-10-11 07:30:07 +0000 |
| commit | c33480057dc35ed3422777ffd68e7cfdcd584a63 (patch) | |
| tree | 20fb98aacb25ced7a459061e910f997fe7d96b57 | |
| parent | Collect status from dead jobs and don't invoke the callback until both (diff) | |
| download | wireguard-openbsd-c33480057dc35ed3422777ffd68e7cfdcd584a63.tar.xz wireguard-openbsd-c33480057dc35ed3422777ffd68e7cfdcd584a63.zip | |
There isn't much point in having a free function if it isn't used.
Also allow a NULL tree.
| -rw-r--r-- | usr.bin/tmux/job.c | 8 |
1 files changed, 6 insertions, 2 deletions
diff --git a/usr.bin/tmux/job.c b/usr.bin/tmux/job.c index 23b183d4094..2e94e3b2961 100644 --- a/usr.bin/tmux/job.c +++ b/usr.bin/tmux/job.c @@ -1,4 +1,4 @@ -/* $OpenBSD: job.c,v 1.3 2009/10/11 07:20:16 nicm Exp $ */ +/* $OpenBSD: job.c,v 1.4 2009/10/11 07:30:07 nicm Exp $ */ /* * Copyright (c) 2009 Nicholas Marriott <nicm@users.sourceforge.net> @@ -107,7 +107,8 @@ job_add(struct jobs *jobs, struct client *c, const char *cmd, job->flags = JOB_DONE; - RB_INSERT(jobs, jobs, job); + if (jobs != NULL) + RB_INSERT(jobs, jobs, job); SLIST_INSERT_HEAD(&all_jobs, job, lentry); return (job); @@ -121,6 +122,9 @@ job_free(struct job *job) xfree(job->cmd); + if (job->freefn != NULL && job->data != NULL) + job->freefn(job->data); + if (job->fd != -1) close(job->fd); if (job->out != NULL) |