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

Uninitialized data access issue fixed #37

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

olesHolem
Copy link

What ❔

This PR adds data_is_set field to structures AsyncVec and DeviceBuf to track if inner data is correct and defined.

Why ❔

Currently, it's possible to get access to uninitialized data. The simplest way is through allocate_new and get_values method of AsyncVec.

Copy link

Hey there! 👋🏼

We require pull request titles to follow the Conventional Commits specification and it looks like your proposed title needs to be adjusted.
Examples of valid PR titles:

  • feat(boojum-cuda): Expose more public methods
  • fix(gpu-ffi): Correctly handle edge case
  • ci: Add new workflow for linting

Details:

No release type found in pull request title "Uninitialized data access issue fixed". Add a prefix to indicate what kind of release this pull request corresponds to. For reference, see https://www.conventionalcommits.org/

Available types:
 - feat: A new feature
 - fix: A bug fix
 - docs: Documentation only changes
 - style: Changes that do not affect the meaning of the code (white-space, formatting, missing semi-colons, etc)
 - refactor: A code change that neither fixes a bug nor adds a feature
 - perf: A code change that improves performance
 - test: Adding missing tests or correcting existing tests
 - build: Changes that affect the build system or external dependencies (example scopes: gulp, broccoli, npm)
 - ci: Changes to our CI configuration files and scripts (example scopes: Travis, Circle, BrowserStack, SauceLabs)
 - chore: Other changes that don't modify src or test files
 - revert: Reverts a previous commit

@robik75
Copy link
Member

robik75 commented Jan 9, 2025

@olesHolem Do you still want to include merge this PR or should we close it?

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.

2 participants