Change in osmocom-bb[master]: trx_toolkit/burst_send.py: exit if DATA dump parsing failed

[Vadim Yanitskiy]

Vadim Yanitskiy has uploaded this change for review. ( https://gerrit.osmocom.org/12324


Change subject: trx_toolkit/burst_send.py: exit if DATA dump parsing failed
......................................................................

trx_toolkit/burst_send.py: exit if DATA dump parsing failed

False is not iterable, so we should properly handle the case
when parsing of the whole DATA dump was failed (e.g. due to
incorrect offset specified).

Change-Id: I5443efb39bb9d3377290ce7ec5e34016cae0edb2
---
M src/target/trx_toolkit/burst_send.py
1 file changed, 2 insertions(+), 1 deletion(-)



  git pull ssh://gerrit.osmocom.org:29418/osmocom-bb refs/changes/24/12324/1

diff --git a/src/target/trx_toolkit/burst_send.py b/src/target/trx_toolkit/burst_send.py
index 7186863..6c19e6f 100755
--- a/src/target/trx_toolkit/burst_send.py
+++ b/src/target/trx_toolkit/burst_send.py
@@ -63,7 +63,8 @@
 		messages = self.ddf.parse_all(
 			skip = self.argv.cnt_skip, count = self.argv.cnt_count)
 		if messages is False:
-			pass # FIXME!!!
+			log.error("Parsing failed, nothing to send")
+			sys.exit(1)
 
 		for msg in messages:
 			# Pass filter

-- 
To view, visit https://gerrit.osmocom.org/12324
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings

Gerrit-Project: osmocom-bb
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I5443efb39bb9d3377290ce7ec5e34016cae0edb2
Gerrit-Change-Number: 12324
Gerrit-PatchSet: 1
Gerrit-Owner: Vadim Yanitskiy <axilirator at gmail.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20181216/54b68b0e/attachment.htm>