diff options
| author | Sumit Bose <sbose@redhat.com> | 2009-06-23 14:57:41 +0200 |
|---|---|---|
| committer | Stephen Gallagher <sgallagh@redhat.com> | 2009-07-02 08:59:48 -0400 |
| commit | 6dc2131ab0a5225decdcdb8fff8706a340d9ed3b (patch) | |
| tree | 4fa5d1c73199081dca916f0883df0200b29e62a9 /server/providers | |
| parent | db89f7616f3595aa2900a55039ee3b4e994eb6dc (diff) | |
| download | sssd-6dc2131ab0a5225decdcdb8fff8706a340d9ed3b.tar.gz sssd-6dc2131ab0a5225decdcdb8fff8706a340d9ed3b.tar.bz2 sssd-6dc2131ab0a5225decdcdb8fff8706a340d9ed3b.zip | |
check pending_return after dbus_connection_send_with_reply
Diffstat (limited to 'server/providers')
| -rw-r--r-- | server/providers/data_provider.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/server/providers/data_provider.c b/server/providers/data_provider.c index e8f190ea..677f677f 100644 --- a/server/providers/data_provider.c +++ b/server/providers/data_provider.c @@ -255,7 +255,7 @@ static int dbus_dp_init(struct sbus_conn_ctx *conn_ctx, void *data) } dbret = dbus_connection_send_with_reply(conn, msg, &pending_reply, 600000 /* TODO: set timeout */); - if (!dbret) { + if (!dbret || pending_reply == NULL) { /* * Critical Failure * We can't communicate on this connection @@ -536,7 +536,7 @@ static int dp_send_acct_req(struct dp_be_request *bereq, ret = dbus_connection_send_with_reply(conn, msg, &pending_reply, 600000 /* TODO: set timeout */); - if (!ret) { + if (!ret || pending_reply == NULL) { /* * Critical Failure * We can't communicate on this connection @@ -830,7 +830,7 @@ static int dp_call_pamhandler(struct dp_be_request *bereq, struct pam_data *pd) ret = dbus_connection_send_with_reply(conn, msg, &pending_reply, 600000 /* TODO: set timeout */); - if (!ret) { + if (!ret || pending_reply == NULL) { /* * Critical Failure * We can't communicate on this connection |