add err checks; fix deref null ptrs, mem boundary/leak issues, etc.

luxonis / XLink

A cross-platform library for communicating with devices over various physical links.

Apache License 2.0

11 stars 17 forks source link

add err checks; fix deref null ptrs, mem boundary/leak issues, etc. #36

Closed diablodale closed 2 years ago

diablodale commented 2 years ago

mostly fix luxonis/XLink#35

found with msvc code analysis
one open concern about bulk_chunklen var as described in linked issue

diablodale commented 2 years ago

Is there a way for CI to build XLink PRs? At a minimum, it would tell if the build is broken. I don't have the harness to adequately build+test your matrix of linux.

Meanwhile, I'll work on the bulk_chunklen changes.

themarpe commented 2 years ago

I've started the CI - will lift this for next PRs for XLink