-
Notifications
You must be signed in to change notification settings - Fork 2
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
Fix possible buffer overflow in ClpSimplexOther
#1
base: stable/1.17
Are you sure you want to change the base?
Conversation
- Fix super build (e.g. FetchContent) integration - cmake: option() honors normal variables (CMP0077)
07ab323
to
6138e74
Compare
ClpSimplexOther::parametrics
ClpSimplexOther
6138e74
to
0b3c287
Compare
P.S. I've noticed this warning when was trying to build https://github.com/google/or-tools |
To be sure, is it the backport of: ps: thx for contributing to FOSS ! |
Yes, it's same fix applied to multiple reps. I initially noticed warning about that when was building coin-or from google's repo. And your repo was used directly. So decided to post it here as well. IDK if upstream would accept patch soon. |
and this line
was added, as same fix is in using second variable. as "POSIX standard forbids strings to overlap." |
7c9fbcf
to
37a4f58
Compare
I have found this as warning from GCC when was installing some packages from AUR.
There is possibility of buffer overflow, especially in this case:
Because we are using content of array
line
inside template, and it can be 199 chars already.