search

mildsunrise / node_netlink

⚒ Use Netlink from Node.js
MIT License
22 stars 2 forks source link

Incorrect length checking in formatAttribute #2

Closed dhl closed 4 years ago

dhl commented 4 years ago

Hi @mildsunrise! Thanks for making this awesome library.

I tried using formatAttribute, but was not able to make it work because the length check seems to have been flipped:

https://github.com/mildsunrise/node_netlink/blob/48099d7f28c779d76f94bc0b58a98ff758ea700e/lib/structs.ts#L274-L279

Would you mind taking a look? Thanks again!

mildsunrise commented 4 years ago

wow, thanks for the heads up!! I didn't notice because I was using the AttrStream API :sweat_smile:

mildsunrise commented 4 years ago

0.1.4 is up :)