Skip to content

Commit

Permalink
Changes as suggested by review:
Browse files Browse the repository at this point in the history
 set err for connection to EINVAL
 cleanups
  • Loading branch information
John Irwin committed Oct 18, 2013
1 parent 40872af commit ad90f09
Showing 1 changed file with 5 additions and 3 deletions.
8 changes: 5 additions & 3 deletions src/nc_response.c
Original file line number Diff line number Diff line change
Expand Up @@ -159,10 +159,12 @@ rsp_filter(struct context *ctx, struct conn *conn, struct msg *msg)
msg->id, msg->mlen, conn->sd);
rsp_put(msg);

/* this can happen as a result of some server errors like object-too-big.
closing the connection is the safest response. */
/*
* This can happen as a result of some server errors like object-too-big.
* closing the connection is the safest response.
*/
conn->err = EINVAL;
conn->done = 1;
log_debug(LOG_NOTICE, "s %d is done", conn->sd);
return true;
}
ASSERT(pmsg->peer == NULL);
Expand Down

0 comments on commit ad90f09

Please sign in to comment.