Skip to content

Fix null-pointer dereference in FieldGetByID#111

Merged
nikkon-dev merged 1 commit intomainfrom
nikkon-dev/fix_field_meta
Feb 10, 2026
Merged

Fix null-pointer dereference in FieldGetByID#111
nikkon-dev merged 1 commit intomainfrom
nikkon-dev/fix_field_meta

Conversation

@nikkon-dev
Copy link
Collaborator

  • Change FieldGetByID to return (FieldMeta, error) and report DCGM failures
  • Properly handle NULL pointers and zeroed structs from the DCGM C API
  • Add GetAllSupportedFieldsMetadata for convenience

- Change FieldGetByID to return (FieldMeta, error) and report DCGM failures
- Properly handle NULL pointers and zeroed structs from the DCGM C API
- Add GetAllSupportedFieldsMetadata for convenience

Signed-off-by: Nik Konyuchenko <spaun2002mobile@gmail.com>
Copy link
Collaborator

@glowkey glowkey left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Collaborator

@nvvfedorov nvvfedorov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@nikkon-dev nikkon-dev merged commit e11a7e2 into main Feb 10, 2026
1 check passed
@nikkon-dev nikkon-dev deleted the nikkon-dev/fix_field_meta branch February 10, 2026 22:19
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