diff options
| author | Yuichi Ito <ito.yuichi0@gmail.com> | 2013-09-25 13:59:01 +0900 |
|---|---|---|
| committer | FUJITA Tomonori <fujita.tomonori@lab.ntt.co.jp> | 2013-09-26 04:02:37 +0900 |
| commit | e7cdb5945775c5807dcb62f01fbcd96436b0ac0c (patch) | |
| tree | a2d2905efd05cc5af1d81ec58b93184db42323ff | |
| parent | 96afdc7bbc13935a83fbba7e796667c6ede712b7 (diff) | |
packet lib: ipv6: avoid dangerous default value [] as argument
Signed-off-by: itoyuichi <ito.yuichi0@gmail.com>
Signed-off-by: FUJITA Tomonori <fujita.tomonori@lab.ntt.co.jp>
| -rw-r--r-- | ryu/lib/packet/ipv6.py | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/ryu/lib/packet/ipv6.py b/ryu/lib/packet/ipv6.py index ca5793d1..4131491e 100644 --- a/ryu/lib/packet/ipv6.py +++ b/ryu/lib/packet/ipv6.py @@ -68,7 +68,7 @@ class ipv6(packet_base.PacketBase): return _register_header_type def __init__(self, version, traffic_class, flow_label, payload_length, - nxt, hop_limit, src, dst, ext_hdrs=[]): + nxt, hop_limit, src, dst, ext_hdrs=None): super(ipv6, self).__init__() self.version = version self.traffic_class = traffic_class @@ -78,10 +78,10 @@ class ipv6(packet_base.PacketBase): self.hop_limit = hop_limit self.src = src self.dst = dst - if ext_hdrs: - assert isinstance(ext_hdrs, list) - for ext_hdr in ext_hdrs: - assert isinstance(ext_hdr, header) + ext_hdrs = ext_hdrs or [] + assert isinstance(ext_hdrs, list) + for ext_hdr in ext_hdrs: + assert isinstance(ext_hdr, header) self.ext_hdrs = ext_hdrs @classmethod |