[PATCH] openbsc[master]: vty tests: testBSCreload: ipa_handle_small: ensure rx of 4 b...

[Neels Hofmeyr]

Hello Jenkins Builder, Holger Freyther,

I'd like you to reexamine a change.  Please visit

    https://gerrit.osmocom.org/1736

to look at the new patch set (#2).

vty tests: testBSCreload: ipa_handle_small: ensure rx of 4 bytes

Change-Id: I995b7aba9d0b51b0861a4281dbbca888f36e9e00
---
M openbsc/tests/vty_test_runner.py
1 file changed, 2 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.osmocom.org:29418/openbsc refs/changes/36/1736/2

diff --git a/openbsc/tests/vty_test_runner.py b/openbsc/tests/vty_test_runner.py
index 3fbe20d..64fd86d 100644
--- a/openbsc/tests/vty_test_runner.py
+++ b/openbsc/tests/vty_test_runner.py
@@ -1239,6 +1239,8 @@
 
 def ipa_handle_small(x, verbose = False):
     s = data2str(x.recv(4))
+    if len(s) != 4*2:
+      raise Exception("expected to receive 4 bytes, but got %d (%r)" % (len(s)/2, s))
     if "0001fe00" == s:
         if (verbose):
             print "\tBSC <- NAT: PING?"

-- 
To view, visit https://gerrit.osmocom.org/1736
To unsubscribe, visit https://gerrit.osmocom.org/settings

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I995b7aba9d0b51b0861a4281dbbca888f36e9e00
Gerrit-PatchSet: 2
Gerrit-Project: openbsc
Gerrit-Branch: master
Gerrit-Owner: Neels Hofmeyr <nhofmeyr at sysmocom.de>
Gerrit-Reviewer: Holger Freyther <holger at freyther.de>
Gerrit-Reviewer: Jenkins Builder