URL: https://github.com/SSSD/sssd/pull/121 Author: fidencio Title: #121: AUTOFS: Check return of sss_cmd_empty_packet() Action: opened
PR body: """ Just to make the static analyzer happier. Resolves: https://fedorahosted.org/sssd/ticket/3275 Signed-off-by: Fabiano FidĂȘncio <fiden...@redhat.com> """ To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/121/head:pr121 git checkout pr121
From 6ff7ccb1b90399dcca4f169e2815601d5565845c Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Fabiano=20Fid=C3=AAncio?= <fiden...@redhat.com> Date: Thu, 12 Jan 2017 01:34:42 +0100 Subject: [PATCH] AUTOFS: Check return of sss_cmd_empty_packet() MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Just to make the static analyzer happier. Resolves: https://fedorahosted.org/sssd/ticket/3275 Signed-off-by: Fabiano FidĂȘncio <fiden...@redhat.com> --- src/responder/autofs/autofssrv_cmd.c | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/src/responder/autofs/autofssrv_cmd.c b/src/responder/autofs/autofssrv_cmd.c index 0878707..ec3d2f4 100644 --- a/src/responder/autofs/autofssrv_cmd.c +++ b/src/responder/autofs/autofssrv_cmd.c @@ -320,7 +320,12 @@ static void sss_autofs_cmd_setautomntent_done(struct tevent_req *req) if (reqret == ENOENT) { DEBUG(SSSDBG_TRACE_FUNC, "setautomntent did not find requested map\n"); /* Notify the caller that this entry wasn't found */ - sss_cmd_empty_packet(pctx->creq->out); + ret = sss_cmd_empty_packet(pctx->creq->out); + if (ret != EOK) { + DEBUG(SSSDBG_TRACE_INTERNAL, + "sss_cmd_empty_packet() failed: %s [%d]\n", + sss_strerror(ret), ret); + } } else { DEBUG(SSSDBG_TRACE_FUNC, "setautomntent found data\n"); ret = sss_packet_grow(pctx->creq->out, 2*sizeof(uint32_t));
_______________________________________________ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org