diff options
| author | Simon McVittie <smcv@collabora.com> | 2019-04-17 14:22:15 +0100 |
|---|---|---|
| committer | Simon McVittie <smcv@collabora.com> | 2019-04-17 14:22:15 +0100 |
| commit | 9e0fd2fbbfd1a86f9c6cf6d909054b806aabde36 (patch) | |
| tree | dfdbb22c39bc33aad88d9814f73d2194b99b7c65 /bus/driver.c | |
| parent | 301db925765e4d95a1876c1fec5c028c5bcd8fe5 (diff) | |
| download | dbus-9e0fd2fbbfd1a86f9c6cf6d909054b806aabde36.tar.gz | |
bus_driver_credentials_fill_unix_gids: Comment on a non-error
As described in the spec, we want GetConnectionCredentials() to report
less information, successfully, if it sees a group ID that it can't
represent.
Signed-off-by: Simon McVittie <smcv@collabora.com>
Diffstat (limited to 'bus/driver.c')
| -rw-r--r-- | bus/driver.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/bus/driver.c b/bus/driver.c index 6714fe06..3898cfd6 100644 --- a/bus/driver.c +++ b/bus/driver.c @@ -1939,6 +1939,9 @@ bus_driver_credentials_fill_unix_gids (DBusCredentials *credentials, { if (gids[i] > _DBUS_UINT32_MAX) { + /* At least one gid is unrepresentable, so behave as though + * we didn't know the group IDs at all (not an error, just + * success with less information) */ dbus_free (gids_u32); return TRUE; } |