Skip to content
This repository has been archived by the owner on Aug 19, 2019. It is now read-only.

dlsym: Used without proper error handling. #120

Open
cheako opened this issue Jun 24, 2017 · 0 comments
Open

dlsym: Used without proper error handling. #120

cheako opened this issue Jun 24, 2017 · 0 comments

Comments

@cheako
Copy link

cheako commented Jun 24, 2017

In atw_vulkan.c, at least, dlsym is used as though it's guarantied the symbol being looked up exists. This may not be correct as it assumes that the Vulkan Loader was built and installed correctly.

To be correct dlsym should be wrapped in dlerror calls and given the usage here it may make sense to put this handling into a function that reports error and exits on symbol not found.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant