Attention is currently required from: pespin.
lynxis lazus has posted comments on this change by lynxis lazus. ( https://gerrit.osmocom.org/c/osmo-remsim/+/42200?usp=email )
Change subject: bankd: csv: fix crash on invalid lines ......................................................................
Patch Set 1:
(1 comment)
File src/bankd/bankd_pcsc.c:
https://gerrit.osmocom.org/c/osmo-remsim/+/42200/comment/a4d1e5df_a00b15ba?u... : PS1, Line 105: LOGP(DMAIN, LOGL_ERROR, "B%d:%d: No reader name given. Maybe invalid csv.\n",
looks ike we want to add a LOG define for sn here :)
Yes, but the logging is still a good topic to change things.I would like to do the topic in a different commit, because it is a general problem and this fixes a crash here.
-
lynxis lazus