[taler-exchange] branch master updated: FIXME is not needed, explain why

[gnunet]

This is an automated email from the git hooks/post-receive script.

grothoff pushed a commit to branch master
in repository exchange.

The following commit(s) were added to refs/heads/master by this push:
     new e88ad199 FIXME is not needed, explain why
e88ad199 is described below

commit e88ad199e667a435046ca2d7cc967ba740fa0452
Author: Christian Grothoff <address@hidden>
AuthorDate: Fri Jan 17 15:02:27 2020 +0100

    FIXME is not needed, explain why
---
 src/auditor/taler-auditor.c | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/src/auditor/taler-auditor.c b/src/auditor/taler-auditor.c
index 3a44c55f..8b2c2c37 100644
--- a/src/auditor/taler-auditor.c
+++ b/src/auditor/taler-auditor.c
@@ -1,6 +1,6 @@
 /*
   This file is part of TALER
-  Copyright (C) 2016, 2017, 2018, 2019 Taler Systems SA
+  Copyright (C) 2016-2020 Taler Systems SA
 
   TALER is free software; you can redistribute it and/or modify it under the
   terms of the GNU Affero Public License as published by the Free Software
@@ -1298,7 +1298,8 @@ handle_payback_by_reserve (void *cls,
   /* should be monotonically increasing */
   GNUNET_assert (rowid >= ppr.last_reserve_payback_serial_id);
   ppr.last_reserve_payback_serial_id = rowid + 1;
-  // FIXME: should probably check that denom_pub hashes to this hash code!
+  /* We know that denom_pub matches denom_pub_hash because this
+     is how the SQL statement joined the tables. */
   pr.h_denom_pub = coin->denom_pub_hash;
   pr.purpose.purpose = htonl (TALER_SIGNATURE_WALLET_COIN_PAYBACK);
   pr.purpose.size = htonl (sizeof (pr));

-- 
To stop receiving notification emails like this one, please contact
address@hidden.