Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

APPLE: Buffer binding null pointer checks #2401

Conversation

creijon
Copy link
Contributor

@creijon creijon commented Apr 22, 2023

Description of Change(s)

If an empty USD file was loaded, a null buffer would be passed into the step function and indirect command buffer bindings, causing a segfault. This protects against it.

Fixes Issue(s)

  • Segfault on binding null buffers
  • I have verified that all unit tests pass with the proposed changes
  • I have submitted a signed Contributor License Agreement

@creijon creijon force-pushed the jon/dev/step_func_null_pointer_check branch 3 times, most recently from 2bccb86 to fc642bb Compare April 22, 2023 02:21
@creijon creijon changed the title APPLE: Step function null pointer check APPLE: Buffer binding null pointer checks Apr 22, 2023
@sunyab
Copy link
Contributor

sunyab commented Apr 24, 2023

Filed as internal issue #USD-8265

@creijon creijon force-pushed the jon/dev/step_func_null_pointer_check branch from fc642bb to 383ad76 Compare April 26, 2023 11:18
@creijon creijon force-pushed the jon/dev/step_func_null_pointer_check branch from 383ad76 to dd39326 Compare May 11, 2023 10:04
@creijon creijon force-pushed the jon/dev/step_func_null_pointer_check branch from dd39326 to 44b6643 Compare May 13, 2023 05:45
@pixar-oss pixar-oss merged commit babb27b into PixarAnimationStudios:dev May 19, 2023
@creijon creijon deleted the jon/dev/step_func_null_pointer_check branch May 21, 2023 15:42
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.

3 participants