exchange

commit af6e9f134a6028cec49056a0fc55e4d243229568
parent 23cb1d46a8b12544735fb420318e5c350fa99bf8
Author: Christian Grothoff <christian@grothoff.org>
Date:   Sat,  9 Dec 2017 23:51:49 +0100

fix potential NPE

Diffstat:
M
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
@@ -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}",