Skip to content
This repository has been archived by the owner on Jul 22, 2023. It is now read-only.

calloc limits #119

Merged
merged 15 commits into from
Apr 12, 2021
Merged

calloc limits #119

merged 15 commits into from
Apr 12, 2021

Conversation

DellaBitta
Copy link
Contributor

@DellaBitta DellaBitta commented Apr 6, 2021

Adds overflow protection to __iotc_calloc calls, as well as unit tests for the function.

@google-cla google-cla bot added the cla: yes CLA approved label Apr 6, 2021
Copy link
Contributor

@atigyi atigyi left a comment

Choose a reason for hiding this comment

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

David, thx for jumping on this topic so quick!! Added few comments. I like the utests!! :)

@DellaBitta DellaBitta marked this pull request as ready for review April 8, 2021 21:56
@DellaBitta DellaBitta requested a review from atigyi April 12, 2021 15:01
Copy link
Contributor

@atigyi atigyi left a comment

Choose a reason for hiding this comment

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

LGTM + thx!! 👍

@DellaBitta DellaBitta merged commit 96018ab into master Apr 12, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
cla: yes CLA approved
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants