Prompted by Werner's comment at https://lwn.net/Articles/1056277/ as
I was wondering if GCC's -fanalyzer would fare any better.

I have some more such fixes depending on how this series goes and if they're
desired.

I've tried to restrict to changes where it seems like a general improvement
and skipped a few obvious false positives where a code change couldn't really
be justified.

Sam James (7):
  common: fix -Wdiscarded-qualifiers
  common: fix -Wunused-but-set-variable
  common: fix -Werror=analyzer-deref-before-check
  common: check strchr for null
  g10: check null in assert
  g10: fix uninit use
  g10: fix uninit use

 agent/command.c               | 12 ++++++------
 common/asshelp.c              |  3 ++-
 common/iobuf.c                |  2 +-
 common/server-help.c          |  4 ++--
 common/server-help.h          |  2 +-
 common/ssh-utils.c            |  3 +--
 common/stringhelp.c           |  8 ++++----
 dirmngr/dirmngr.c             |  2 +-
 g10/decrypt-data.c            |  2 +-
 g10/encrypt.c                 |  2 +-
 g10/keyedit.c                 |  4 ++--
 g10/misc.c                    |  2 +-
 g10/tofu.c                    |  2 +-
 sm/gpgsm.c                    |  4 ++--
 tests/asschk.c                |  2 +-
 tests/openpgp/fake-pinentry.c |  8 ++++----
 tools/gpgconf.c               |  2 +-
 17 files changed, 32 insertions(+), 32 deletions(-)

-- 
2.52.0


_______________________________________________
Gnupg-devel mailing list
[email protected]
https://lists.gnupg.org/mailman/listinfo/gnupg-devel

Reply via email to