JoelBender / bacpypes

BACpypes provides a BACnet application layer and network layer written in Python for daemons, scripting, and graphical interfaces.
MIT License
296 stars 128 forks source link

bufferOverflow while reading objectList #373

Open damienpicard opened 3 years ago

damienpicard commented 3 years ago

The apdu length is specified in BACpypes.ini. This might be good for a project but it might be too short to read the objectList (I encountered that problem in one of my projects). It might be useful to skip that check in that case.

I solved it manually by temporary increasing maxApduLengthAccepted.

zoopp commented 3 years ago

You should adapt your read strategy based on the capabilities of the device that you're interacting with (supported services, max apdu length accepted, segmentation and number of segments). In the worst case scenario you might have to go as basic as reading each item individually.

Correct me if I'm wrong but I wouldn't say this is an issue with bacpypes.