diff options
author | Christian Grothoff <christian@grothoff.org> | 2017-10-07 15:10:24 +0200 |
---|---|---|
committer | Christian Grothoff <christian@grothoff.org> | 2017-10-07 15:10:24 +0200 |
commit | cc5d09cf1d28e31ca8eca054b7da2c3873e2efe7 (patch) | |
tree | eb843f228165457fa066038058df380fc5652405 /src/exchange/taler-exchange-httpd_refresh_melt.c | |
parent | 7935349d35611901d42d088f0e6dbd5ec8c47272 (diff) | |
download | exchange-cc5d09cf1d28e31ca8eca054b7da2c3873e2efe7.tar.gz exchange-cc5d09cf1d28e31ca8eca054b7da2c3873e2efe7.tar.bz2 exchange-cc5d09cf1d28e31ca8eca054b7da2c3873e2efe7.zip |
properly handle signing errors if httpd lacks signing keys by returning internal errors (and handling new return value from TEH_KS_sign)
Diffstat (limited to 'src/exchange/taler-exchange-httpd_refresh_melt.c')
-rw-r--r-- | src/exchange/taler-exchange-httpd_refresh_melt.c | 12 |
1 files changed, 9 insertions, 3 deletions
diff --git a/src/exchange/taler-exchange-httpd_refresh_melt.c b/src/exchange/taler-exchange-httpd_refresh_melt.c index c9914f222..320ec9d8b 100644 --- a/src/exchange/taler-exchange-httpd_refresh_melt.c +++ b/src/exchange/taler-exchange-httpd_refresh_melt.c @@ -135,9 +135,15 @@ reply_refresh_melt_success (struct MHD_Connection *connection, body.session_hash = *session_hash; body.noreveal_index = htons (noreveal_index); body.reserved = htons (0); - TEH_KS_sign (&body.purpose, - &pub, - &sig); + if (GNUNET_OK != + TEH_KS_sign (&body.purpose, + &pub, + &sig)) + { + return TEH_RESPONSE_reply_internal_error (connection, + TALER_EC_EXCHANGE_BAD_CONFIGURATION, + "no keys"); + } sig_json = GNUNET_JSON_from_data_auto (&sig); GNUNET_assert (NULL != sig_json); return TEH_RESPONSE_reply_json_pack (connection, |