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

Update ensure_byte_buf_has_allocated_buffer_member to include primary allocation #737

Open
adpaco-aws opened this issue Dec 1, 2020 · 2 comments
Labels
cbmc Anything related to CBMC proofs. feature-request A feature should be added or improved. p3 This is a minor priority issue

Comments

@adpaco-aws
Copy link
Contributor

Update ensure_byte_buf_has_allocated_buffer_member to include primary allocation so that the code

struct aws_byte_buf *buf1 = malloc(sizeof(*buf1));

ensure_byte_buf_has_allocated_buffer_member(buf1);

can be replaced by

struct aws_byte_buf *buf1 = ensure_byte_buf_has_allocated_buffer_member(buf1);
@feliperodri feliperodri added the cbmc Anything related to CBMC proofs. label Dec 7, 2020
@feliperodri feliperodri changed the title Update ensure_byte_buf_has_allocated_buffer_member to include primary allocation Update ensure_byte_buf_has_allocated_buffer_member to include primary allocation Dec 7, 2020
@yasminetalby yasminetalby added feature-request A feature should be added or improved. p3 This is a minor priority issue labels Jun 25, 2023
@jmklix
Copy link
Member

jmklix commented Sep 21, 2023

Has this been added with the merged PR?

@jmklix jmklix added the response-requested Waiting on additional info and feedback. Will move to 'closing-soon' in 2 days. label Sep 21, 2023
@adpaco-aws
Copy link
Contributor Author

Hi @jmklix , which PR are you referring to? The issue comes from this comment and I believe it's still pending.

@github-actions github-actions bot removed the response-requested Waiting on additional info and feedback. Will move to 'closing-soon' in 2 days. label Sep 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cbmc Anything related to CBMC proofs. feature-request A feature should be added or improved. p3 This is a minor priority issue
Projects
None yet
Development

No branches or pull requests

4 participants