diff options
author | Lukas Slebodnik <lslebodn@redhat.com> | 2015-11-19 14:17:36 +0000 |
---|---|---|
committer | Lukas Slebodnik <lslebodn@redhat.com> | 2016-01-29 09:32:18 +0100 |
commit | f47a339d7794cd5a24d368b3b3640452686e45a5 (patch) | |
tree | 21bb397fe92e85348b2c9d3a7862675a79c9bec6 /contrib | |
parent | 2ff8131cf02decaf0dd0754e843732fe7774fc59 (diff) | |
download | sssd-f47a339d7794cd5a24d368b3b3640452686e45a5.tar.gz sssd-f47a339d7794cd5a24d368b3b3640452686e45a5.tar.xz sssd-f47a339d7794cd5a24d368b3b3640452686e45a5.zip |
pyhbac: Fix warning Wsign-compare
src/python/pyhbac.c: In function ‘HbacRuleElement_repr’:
src/python/pyhbac.c:506:59: error: comparison between
signed and unsigned integer expressions [-Werror=sign-compare]
if (strnames == NULL || strgroups == NULL || category == -1) {
^
src/python/pyhbac.c: In function ‘HbacRuleElement_to_native’:
src/python/pyhbac.c:614:51: error: comparison between
signed and unsigned integer expressions [-Werror=sign-compare]
if (!el->names || !el->groups || el->category == -1) {
^
The static function native_category had type of terurn value uint32_t
But it also could return -1 which indicated an error.
It's better to don't mix return code with returned value.
Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
Diffstat (limited to 'contrib')
0 files changed, 0 insertions, 0 deletions