diff options
| author | Sumit Bose <sbose@redhat.com> | 2009-05-14 12:38:03 +0200 |
|---|---|---|
| committer | Simo Sorce <ssorce@redhat.com> | 2009-05-14 11:29:02 -0400 |
| commit | 5422d203f05859fa2841e3a68980813806eb9581 (patch) | |
| tree | d33d35a0cfe135e330c360ea4069022620a1981b /server/providers | |
| parent | fbd46659003e418c2cd3551b425fcd24a1befd13 (diff) | |
| download | sssd-5422d203f05859fa2841e3a68980813806eb9581.tar.gz sssd-5422d203f05859fa2841e3a68980813806eb9581.tar.xz sssd-5422d203f05859fa2841e3a68980813806eb9581.zip | |
added check for NULL values
- allow unspecified value in struct pam_data to be NULL
- check if domain structure is initialized in pam_reply
Diffstat (limited to 'server/providers')
| -rw-r--r-- | server/providers/dp_auth_util.c | 7 |
1 files changed, 7 insertions, 0 deletions
diff --git a/server/providers/dp_auth_util.c b/server/providers/dp_auth_util.c index 630dafe8c..366a3fb2d 100644 --- a/server/providers/dp_auth_util.c +++ b/server/providers/dp_auth_util.c @@ -59,6 +59,13 @@ bool dp_pack_pam_request(DBusMessage *msg, struct pam_data *pd) { int ret; + if (pd->user == NULL || pd->domain == NULL) return false; + if (pd->service == NULL) pd->service = ""; + if (pd->tty == NULL) pd->tty = ""; + if (pd->ruser == NULL) pd->ruser = ""; + if (pd->rhost == NULL) pd->rhost = ""; + + ret = dbus_message_append_args(msg, DBUS_TYPE_INT32, &(pd->cmd), DBUS_TYPE_STRING, &(pd->domain), |