diff options
| author | Sumit Bose <sbose@redhat.com> | 2009-04-23 12:56:08 +0200 |
|---|---|---|
| committer | Stephen Gallagher <sgallagh@redhat.com> | 2009-04-23 09:06:27 -0400 |
| commit | a03f9612e6560d4aa32e2813cce1bad88b550713 (patch) | |
| tree | d30d82efc6df5a784c8b7e41052d046ffa089ca9 | |
| parent | 1e08c6f901ad477d9be4b74942487bb3e8573ce3 (diff) | |
| download | sssd-a03f9612e6560d4aa32e2813cce1bad88b550713.tar.gz sssd-a03f9612e6560d4aa32e2813cce1bad88b550713.tar.xz sssd-a03f9612e6560d4aa32e2813cce1bad88b550713.zip | |
removed length of unused element from packet size calculation
The domain name is no longer send as an element on its own, but
if set as a member of the response array. If the user was not found
pd->domain is NULL and strlen will seg-fault.
| -rw-r--r-- | server/responder/pam/pamsrv_cmd.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/server/responder/pam/pamsrv_cmd.c b/server/responder/pam/pamsrv_cmd.c index 45f44c3e9..158cb32d8 100644 --- a/server/responder/pam/pamsrv_cmd.c +++ b/server/responder/pam/pamsrv_cmd.c @@ -217,7 +217,7 @@ static void pam_reply(struct pam_auth_req *preq) resp = resp->next; } - ret = sss_packet_grow(cctx->creq->out, sizeof(int32_t) + strlen(pd->domain)+1 + + ret = sss_packet_grow(cctx->creq->out, sizeof(int32_t) + sizeof(int32_t) + resp_c * 2* sizeof(int32_t) + resp_size); |