diff options
| author | Stanislav Laznicka <slaznick@redhat.com> | 2016-02-02 12:57:11 +0100 |
|---|---|---|
| committer | Jan Cholasta <jcholast@redhat.com> | 2016-02-24 09:21:30 +0100 |
| commit | d7efd8a33ab14a561d3af445e62bceb6f2f13fd1 (patch) | |
| tree | 2f03d17484383bc4114fff16cc9a5f56883e6752 /ipatests | |
| parent | 9757384c7c6337138b6e096d3bb926cdb3012ff6 (diff) | |
Fixes minor issues
Fixes issues discovered by Coverity
plugins/baseldap.py: possible undefined value in return
certmonger.py: possible dereference of None value
i18n.py: fixed always True bug (+ cosmetic change)
https://fedorahosted.org/freeipa/ticket/5661
Reviewed-By: Jan Cholasta <jcholast@redhat.com>
Reviewed-By: Martin Basti <mbasti@redhat.com>
Diffstat (limited to 'ipatests')
| -rwxr-xr-x | ipatests/i18n.py | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/ipatests/i18n.py b/ipatests/i18n.py index a83c5806e..368712599 100755 --- a/ipatests/i18n.py +++ b/ipatests/i18n.py @@ -755,7 +755,7 @@ def main(): print('ERROR: no mode specified', file=sys.stderr) return 1 - if options.mode == 'validate_pot' or options.mode == 'validate_po': + if options.mode in ('validate_pot', 'validate_po'): if options.mode == 'validate_pot': files = args if not files: @@ -795,7 +795,7 @@ def main(): else: return 0 - elif options.mode == 'create_test' or 'test_gettext': + elif options.mode in ('create_test', 'test_gettext'): po_file = '%s.po' % options.test_lang pot_file = options.pot_file |