From 90413e60306068567d8a207532fdc51785ac4da3 Mon Sep 17 00:00:00 2001 From: Shane Alcock Date: Thu, 19 Jun 2025 02:02:36 -0400 Subject: [PATCH] Fix bug where adding extra options to bsrt input would segfault This was preventing us from setting the broker URL to anything other than the default CAIDA broker. --- lib/io/bsrt/bgpview_io_bsrt.c | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/lib/io/bsrt/bgpview_io_bsrt.c b/lib/io/bsrt/bgpview_io_bsrt.c index 83379ea..3c2f4ee 100644 --- a/lib/io/bsrt/bgpview_io_bsrt.c +++ b/lib/io/bsrt/bgpview_io_bsrt.c @@ -999,10 +999,8 @@ bgpview_io_bsrt_t *bgpview_io_bsrt_init(const char *opts, timeseries_t *timeseri fprintf(stderr, "ERROR: Could not create BGPStream instance\n"); goto err; } - if (bsrt->di_id != 0) { - bsrt->di_id_default = bsrt->di_id = bgpstream_get_data_interface_id(bsrt->stream); - bsrt->di_info = bgpstream_get_data_interface_info(bsrt->stream, bsrt->di_id); - } + bsrt->di_id_default = bsrt->di_id = bgpstream_get_data_interface_id(bsrt->stream); + bsrt->di_info = bgpstream_get_data_interface_info(bsrt->stream, bsrt->di_id); if (opts != NULL && (len = strlen(opts)) > 0) { // parse the option string ready for getopt