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

Add checks to prevent out-of-bounds write on tag->encoded_name #368

Merged

Conversation

tinajohnson
Copy link
Contributor

Fixes #367

@kyle-github
Copy link
Member

Hmm, same CI problem as the other PR.

I think this is a good start, but I think that there might be other areas where this could happen.

@kyle-github kyle-github changed the base branch from release to prerelease July 1, 2022 00:44
@kyle-github
Copy link
Member

Moving to prerelease for CI fix up addition.

@kyle-github kyle-github merged commit 9934575 into libplctag:prerelease Jul 1, 2022
kyle-github added a commit that referenced this pull request Jul 1, 2022
…370)

* Add condition to fail tag creation when elem_count is a negative value. (#366)

Fixes #365

* Change CI build to use Visual Studio 17 2022.  Apparently version 16 is gone on Github?

* Add checks to prevent out-of-bounds write on tag->encoded_name (#368)

Fixes #367

Co-authored-by: Tina Johnson <[email protected]>
kyle-github added a commit that referenced this pull request Sep 24, 2022
…rrect comparison (#383)

* Add condition to fail tag creation when elem_count is a negative value. (#366)

Fixes #365

* Change CI build to use Visual Studio 17 2022.  Apparently version 16 is gone on Github?

* Add checks to prevent out-of-bounds write on tag->encoded_name (#368)

Fixes #367

* Update stable vs. unstable versions.

* Bump version for next release.

* Fix 378.   Refactor condition var signal into common exit path of state SESSION_OPEN_SOCKET_START.

* Fix 381. Incorrect comparison in loop guarding cond_wait from spurious wakeups.

* Add null terminator to tag->name string (#372)

Fixes #371

Co-authored-by: Tina Johnson <[email protected]>
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.

Potential heap buffer overflow in cip.c
2 participants