exabgp.bgp.message.update.attribute.bgpls.link.sradjlan

Complexity

Total Complexity

10

Size/Duplication

Total Lines	93
Duplicated Lines	59.14 %

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	SrAdjacencyLan.json()	2	2	1
A	SrAdjacencyLan.__repr__()	2	2	1
A	SrAdjacencyLan.reset()	3	3	1
A	SrAdjacencyLan.__init__()	2	2	1
B	SrAdjacencyLan.unpack()	37	37	6

# encoding: utf-8
"""
sradjlan.py

Created by Evelio Vila
Copyright (c) 2014-2017 Exa Networks. All rights reserved.
"""

import json
from struct import unpack
from exabgp.vendoring import six
from exabgp.util import hexstring

from exabgp.vendoring.bitstring import BitArray
from exabgp.protocol.iso import ISO
from exabgp.bgp.message.update.attribute.bgpls.linkstate import LINKSTATE, LsGenericFlags
from exabgp.bgp.message.notification import Notify

#   0                   1                   2                   3
#   0 1 2 3 4 5 6 7 8 9 0 1 2 3 4 5 6 7 8 9 0 1 2 3 4 5 6 7 8 9 0 1
#  +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+
#  |              Type             |            Length             |
#  +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+
#  |     Flags     |     Weight    |            Reserved           |
#  +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+
#
#   +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+
#   |             OSPF Neighbor ID / IS-IS System-ID                |
#   +                               +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+
#   |                               |
#   +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+
#
#   +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+
#   |                    SID/Label/Index (variable)                 |
#   +---------------------------------------------------------------+
#		draft-gredler-idr-bgp-ls-segment-routing-ext-03

@LINKSTATE.register()

This code seems to be duplicated in your project.

class SrAdjacencyLan(object):
	TLV = 1100
	_sr_adj_lan_sids = []

	def __init__ (self):
		self.sr_adj_lan_sids = SrAdjacencyLan._sr_adj_lan_sids.copy()

	def __repr__ (self):
		return "sr-adj-lan-sids: {}".format(self.sr_adj_lan_sids)

	@classmethod
	def unpack (cls,data,length):
		# We only support IS-IS flags for now.
		flags = LsGenericFlags.unpack(data[0:1],LsGenericFlags.ISIS_SR_ADJ_FLAGS).flags
		# Parse adj weight
		weight = six.indexbytes(data,1)
		# Parse neighbor System-ID
		system_id = ISO.unpack_sysid(data[4:10])
		# Move pointer 10 bytes: Flags(1) + Weight(1) + Reserved(2) + System-ID(6)
		data = data[10:]
     	# SID/Index/Label: according to the V and L flags, it contains
      	# either:
		# *  A 3 octet local label where the 20 rightmost bits are used for
		#	 encoding the label value.  In this case the V and L flags MUST
		#	 be set.
		#
		# *  A 4 octet index defining the offset in the SID/Label space
		# 	 advertised by this router using the encodings defined in
		#  	 Section 3.1.  In this case V and L flags MUST be unset.
		raw = []
		while data:
			# Range Size: 3 octet value indicating the number of labels in
			# the range.
			if int(flags['V']) and int(flags['L']):
				b = BitArray(bytes=data[:3])
				sid = b.unpack('uintbe:24')[0]
				data = data[3:]
			elif (not flags['V']) and (not flags['L']):
				sid = unpack('!I',data[:4])[0]
				data = data[4:]
			else:
				raw.append(hexstring(data))
				break
		cls._sr_adj_lan_sids.append(
			{'flags': flags, 'weight': weight, 'system-id': system_id, 'sid': sid, 'undecoded': raw}

The variable sid does not seem to be defined in case the while loop on line 69 is not entered. Are you sure this can never be the case?

		)
		return cls()

	def json (self,compact=None):
		return '"sr-adj-lan-sids": {}'.format(json.dumps(self.sr_adj_lan_sids))

	@classmethod
	def reset(cls):
		cls._sr_adj_sids = []