From cacb69b4e10dec2f55136b069be971f7d6e3cb42 Mon Sep 17 00:00:00 2001 From: Minoru TAKAHASHI Date: Wed, 30 Jul 2014 14:31:46 +0900 Subject: packet lib: ipv4: fix default arguments Reported-by: Arne Goetje Signed-off-by: Minoru TAKAHASHI Signed-off-by: FUJITA Tomonori --- ryu/lib/packet/ipv4.py | 4 ++-- ryu/tests/unit/packet/test_packet.py | 20 ++++++++++---------- 2 files changed, 12 insertions(+), 12 deletions(-) diff --git a/ryu/lib/packet/ipv4.py b/ryu/lib/packet/ipv4.py index a252c70c..4da0bccc 100644 --- a/ryu/lib/packet/ipv4.py +++ b/ryu/lib/packet/ipv4.py @@ -79,8 +79,8 @@ class ipv4(packet_base.PacketBase): def __init__(self, version=4, header_length=5, tos=0, total_length=0, identification=0, flags=0, offset=0, ttl=255, proto=0, csum=0, - src='0.0.0.0', - dst='0.0.0.0', + src='10.0.0.1', + dst='10.0.0.2', option=None): super(ipv4, self).__init__() self.version = version diff --git a/ryu/tests/unit/packet/test_packet.py b/ryu/tests/unit/packet/test_packet.py index b4eacd6f..891cbf12 100644 --- a/ryu/tests/unit/packet/test_packet.py +++ b/ryu/tests/unit/packet/test_packet.py @@ -630,11 +630,11 @@ class TestPacket(unittest.TestCase): eq_(0, p_ipv4.flags) eq_(255, p_ipv4.ttl) eq_(inet.IPPROTO_SCTP, p_ipv4.proto) - eq_('0.0.0.0', p_ipv4.src) - eq_('0.0.0.0', p_ipv4.dst) + eq_('10.0.0.1', p_ipv4.src) + eq_('10.0.0.2', p_ipv4.dst) t = bytearray(ip_buf) struct.pack_into('!H', t, 10, p_ipv4.csum) - eq_(packet_utils.checksum(t), 0) + eq_(packet_utils.checksum(t), 0x1403) # sctp ok_(p_sctp) @@ -673,8 +673,8 @@ class TestPacket(unittest.TestCase): 'ttl': 255, 'proto': inet.IPPROTO_SCTP, 'csum': p_ipv4.csum, - 'src': '0.0.0.0', - 'dst': '0.0.0.0', + 'src': '10.0.0.1', + 'dst': '10.0.0.2', 'option': None} _ipv4_str = ','.join(['%s=%s' % (k, repr(ipv4_values[k])) for k, v in inspect.getmembers(p_ipv4) @@ -779,11 +779,11 @@ class TestPacket(unittest.TestCase): eq_(0, p_ipv4.flags) eq_(255, p_ipv4.ttl) eq_(inet.IPPROTO_ICMP, p_ipv4.proto) - eq_('0.0.0.0', p_ipv4.src) - eq_('0.0.0.0', p_ipv4.dst) + eq_('10.0.0.1', p_ipv4.src) + eq_('10.0.0.2', p_ipv4.dst) t = bytearray(ip_buf) struct.pack_into('!H', t, 10, p_ipv4.csum) - eq_(packet_utils.checksum(t), 0) + eq_(packet_utils.checksum(t), 0x1403) # icmp ok_(p_icmp) @@ -815,8 +815,8 @@ class TestPacket(unittest.TestCase): 'ttl': 255, 'proto': inet.IPPROTO_ICMP, 'csum': p_ipv4.csum, - 'src': '0.0.0.0', - 'dst': '0.0.0.0', + 'src': '10.0.0.1', + 'dst': '10.0.0.2', 'option': None} _ipv4_str = ','.join(['%s=%s' % (k, repr(ipv4_values[k])) for k, _ in inspect.getmembers(p_ipv4) -- cgit v1.2.3