Free only fake jobs.

If the job has an id, it is a "real" job and should only be freed by a delete command.
echou · Feb 23, 2009 · fe3256e · fe3256e
1 parent 3427936
commit fe3256e
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/conn.c b/conn.c
@@ -271,7 +271,7 @@ conn_close(conn c)
  job_free(c->in_job);
 
  /* was this a peek or stats command? */
- if (!has_reserved_this_job(c, c->out_job)) job_free(c->out_job);
+ if (c->out_job && !c->out_job->id) job_free(c->out_job);
 
  c->in_job = c->out_job = NULL;
  c->in_job_read = 0;

diff --git a/prot.c b/prot.c
@@ -1426,7 +1426,7 @@ reset_conn(conn c)
  if (r == -1) return twarnx("update events failed"), conn_close(c);
 
  /* was this a peek or stats command? */
- if (!has_reserved_this_job(c, c->out_job)) job_free(c->out_job);
+ if (c->out_job && !c->out_job->id) job_free(c->out_job);
  c->out_job = NULL;
 
  c->reply_sent = 0; /* now that we're done, reset this */