From 72c5b2d7966707c97ceedce04cad6dff065fa3c0 Mon Sep 17 00:00:00 2001 From: Harald Welte Date: Sun, 24 Jul 2022 10:21:41 +0200 Subject: [PATCH] pySim-trace: Fix --no-suppress-{select.status} command line arguments The Tracer implemented those options and the argparser handled it, but we didn't ever connect the two. Change-Id: I7d7d5fc475a8d09efdb63d3d6f1cc1de1996687b --- pySim-trace.py | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/pySim-trace.py b/pySim-trace.py index 8ee82f32..c480168a 100755 --- a/pySim-trace.py +++ b/pySim-trace.py @@ -117,8 +117,10 @@ option_parser = argparse.ArgumentParser(prog='pySim-trace', description='Osmocom formatter_class=argparse.ArgumentDefaultsHelpFormatter) global_group = option_parser.add_argument_group('General Options') -global_group.add_argument('--no-suppress-select', help="Don't suppress displaying SELECT APDUs") -global_group.add_argument('--no-suppress-status', help="Don't suppress displaying STATUS APDUs") +global_group.add_argument('--no-suppress-select', action='store_false', dest='suppress_select', + help="Don't suppress displaying SELECT APDUs") +global_group.add_argument('--no-suppress-status', action='store_false', dest='suppress_status', + help="Don't suppress displaying STATUS APDUs") subparsers = option_parser.add_subparsers(help='APDU Source', dest='source', required=True) @@ -153,7 +155,7 @@ if __name__ == '__main__': elif opts.source == 'rspro-pyshark-live': s = PysharkRsproLive(opts.interface) - tracer = Tracer(source=s) + tracer = Tracer(source=s, suppress_status=opts.suppress_status, suppress_select=opts.suppress_select) logger.info('Entering main loop...') tracer.main()