From dfc33ddcfd41b0d44471acdd7395356273b54d0f Mon Sep 17 00:00:00 2001 From: Jakub Hrozek Date: Tue, 10 May 2016 12:24:44 +0200 Subject: [PATCH 2/2] IFP: Do not crash on invalid arguments to GetUserAttr --- src/responder/ifp/ifpsrv_cmd.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/responder/ifp/ifpsrv_cmd.c b/src/responder/ifp/ifpsrv_cmd.c index 399e83e0255700df5a24491a3eb33b4f92040ca7..2c0ceb817154b7cb60d2d65637bfebd3e738992f 100644 --- a/src/responder/ifp/ifpsrv_cmd.c +++ b/src/responder/ifp/ifpsrv_cmd.c @@ -83,7 +83,7 @@ int ifp_user_get_attr(struct sbus_request *dbus_req, void *data) ret = ifp_user_get_attr_unpack_msg(attr_req); if (ret != EOK) { - return ret; /* handled internally */ + return ret; /* internal error */ } DEBUG(SSSDBG_FUNC_DATA, @@ -117,7 +117,7 @@ ifp_user_get_attr_unpack_msg(struct ifp_attr_req *attr_req) DBUS_TYPE_INVALID); if (parsed == false) { DEBUG(SSSDBG_OP_FAILURE, "Could not parse arguments\n"); - return EOK; /* handled */ + return ERR_SBUS_REQUEST_HANDLED; } /* Copy the attributes to maintain memory hierarchy with talloc */ -- 2.4.11