Skip to content

Fix null pointer reference issue with cuda driver API function pointer…#379

Merged
rwarmstr merged 8 commits intoNVIDIA:masterfrom
nt-kar:bug_matrixmulDynlinkJIT_crash
Sep 5, 2025
Merged

Fix null pointer reference issue with cuda driver API function pointer…#379
rwarmstr merged 8 commits intoNVIDIA:masterfrom
nt-kar:bug_matrixmulDynlinkJIT_crash

Conversation

@nt-kar
Copy link
Copy Markdown
Contributor

@nt-kar nt-kar commented Aug 1, 2025

We see app crash when CuInit returns failure before initializing all the function pointers. Error path should not crash. PR for failsafe error path.

@nt-kar nt-kar closed this Aug 5, 2025
@nt-kar
Copy link
Copy Markdown
Contributor Author

nt-kar commented Aug 5, 2025

updates

@nt-kar nt-kar reopened this Aug 5, 2025
Comment thread Samples/0_Introduction/matrixMulDynlinkJIT/cuda_drvapi_dynlink.c
Comment thread Samples/0_Introduction/matrixMulDynlinkJIT/cuda_drvapi_dynlink.c Outdated
nt-kar added 2 commits August 6, 2025 00:29
… with error in case of LOADLIBRARY failureas initialize of function pointers in case of LOADLIBRARY failure will fail
Comment thread Samples/0_Introduction/matrixMulDynlinkJIT/cuda_drvapi_dynlink.c Outdated
Comment thread Samples/0_Introduction/matrixMulDynlinkJIT/cuda_drvapi_dynlink.c Outdated
Comment thread Samples/0_Introduction/matrixMulDynlinkJIT/cuda_drvapi_dynlink.c Outdated
Comment thread Samples/0_Introduction/matrixMulDynlinkJIT/cuda_drvapi_dynlink.c Outdated
Comment thread Samples/0_Introduction/matrixMulDynlinkJIT/cuda_drvapi_dynlink.c Outdated
@nt-kar nt-kar changed the title Fix null pointer refrence issue with cuda driver API function pointer… Fix null pointer reference issue with cuda driver API function pointer… Aug 8, 2025
@rwarmstr
Copy link
Copy Markdown
Collaborator

Thanks for submitting this pull request! Functionally looks good, but whitespace and code formatting are not correct for this repository's style guidelines. Can you please run pre-commit against your changes?

@nt-kar
Copy link
Copy Markdown
Contributor Author

nt-kar commented Aug 29, 2025

Thanks for submitting this pull request! Functionally looks good, but whitespace and code formatting are not correct for this repository's style guidelines. Can you please run pre-commit against your changes?

fixed

@rwarmstr
Copy link
Copy Markdown
Collaborator

rwarmstr commented Sep 5, 2025

Looks good, thanks for your submission. Merging.

@rwarmstr rwarmstr merged commit c94ff36 into NVIDIA:master Sep 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants