diff options
| author | Todd Zullinger <tmz@pobox.com> | 2021-11-26 09:35:21 +0100 |
|---|---|---|
| committer | Werner Koch <wk@gnupg.org> | 2022-04-25 12:09:50 +0200 |
| commit | beb79f2705ad24665f54c9ea2042a73b1f16f6bc (patch) | |
| tree | 8976e5ea7bfb84eab9f545e4fc5d5775ff1a95a0 /sm | |
| parent | Deprecate the --supervised options. (diff) | |
| download | gnupg2-beb79f2705ad24665f54c9ea2042a73b1f16f6bc.tar.xz gnupg2-beb79f2705ad24665f54c9ea2042a73b1f16f6bc.zip | |
sm: Use gpg_err_code() instead of -1
* sm/verify.c (gpgsm_verify): use gpg_err_code instead of 'rc == -1'
comparison.
--
In ed6ebb696e (sm: Implement initial support for keyboxd., 2020-09-10),
the return of keydb_search() was changed to use gpg_err_code(). Adjust
gpgsm_verify() to use it. This provides a nicer error message to users
when a certificate is not found -- the same error that is seen from 2.2.
Prior to this change, when a cert was not found, gpgsm output:
gpgsm: failed to find the certificate: Not found
as opposed to the more human-readable message from 2.2:
gpgsm: certificate not found
They now return the same message.
Signed-off-by: Todd Zullinger <tmz@pobox.com>
Diffstat (limited to 'sm')
| -rw-r--r-- | sm/verify.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/sm/verify.c b/sm/verify.c index c9a435895..10d5d5c35 100644 --- a/sm/verify.c +++ b/sm/verify.c @@ -428,7 +428,7 @@ gpgsm_verify (ctrl_t ctrl, int in_fd, int data_fd, estream_t out_fp) rc = keydb_search_issuer_sn (ctrl, kh, issuer, serial); if (rc) { - if (rc == -1) + if (gpg_err_code (rc) == GPG_ERR_NOT_FOUND) { log_error ("certificate not found\n"); rc = gpg_error (GPG_ERR_NO_PUBKEY); |