[GNUnet-SVN] [taler-exchange] branch master updated: fix potential NPE

[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 af6e9f1  fix potential NPE
af6e9f1 is described below

commit af6e9f134a6028cec49056a0fc55e4d243229568
Author: Christian Grothoff <address@hidden>
AuthorDate: Sat Dec 9 23:51:49 2017 +0100

    fix potential NPE
---
 src/auditor/taler-auditor.c | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/src/auditor/taler-auditor.c b/src/auditor/taler-auditor.c
index 390e015..7ccb9e8 100644
--- a/src/auditor/taler-auditor.c
+++ b/src/auditor/taler-auditor.c
@@ -1073,6 +1073,9 @@ handle_payback_by_reserve (void *cls,
       report_row_inconsistency ("payback",
                                rowid,
                                "denomination key not in revocation set");
+      /* FIXME: add amount involved to some loss statistic!?
+         It's kind-of not a loss (we just paid back), OTOH, it is
+         certainly irregular and involves some amount.  */
     }
     else
     {
@@ -1105,7 +1108,8 @@ handle_payback_by_reserve (void *cls,
   {
     rev_rowid = 0; /* reported elsewhere */
   }
-  if (0 == strcmp (rev, "master signature invalid"))
+  if ( (NULL != rev) &&
+       (0 == strcmp (rev, "master signature invalid")) )
   {
     report (report_bad_sig_losses,
             json_pack ("{s:s, s:I, s:o, s:o}",

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