diff options
| author | Christian Grothoff <christian@grothoff.org> | 2017-12-09 23:51:49 +0100 |
|---|---|---|
| committer | Christian Grothoff <christian@grothoff.org> | 2017-12-09 23:51:49 +0100 |
| commit | af6e9f134a6028cec49056a0fc55e4d243229568 (patch) | |
| tree | 8f20d33c83c85058ae238a3c08a3d34ee560d867 /src/auditor | |
| parent | 23cb1d46a8b12544735fb420318e5c350fa99bf8 (diff) | |
| download | exchange-af6e9f134a6028cec49056a0fc55e4d243229568.tar.gz exchange-af6e9f134a6028cec49056a0fc55e4d243229568.tar.bz2 exchange-af6e9f134a6028cec49056a0fc55e4d243229568.zip | |
fix potential NPE
Diffstat (limited to 'src/auditor')
| -rw-r--r-- | src/auditor/taler-auditor.c | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/src/auditor/taler-auditor.c b/src/auditor/taler-auditor.c index 390e015c9..7ccb9e897 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}", |