Bug 397531
| Summary: | Freeciv FORTIFY failure during autosave | ||
|---|---|---|---|
| Product: | [Fedora] Fedora | Reporter: | Jerry James <loganjerry> |
| Component: | freeciv | Assignee: | Brian Pepple <bdpepple> |
| Status: | CLOSED ERRATA | QA Contact: | Fedora Extras Quality Assurance <extras-qa> |
| Severity: | medium | Docs Contact: | |
| Priority: | medium | ||
| Version: | 8 | ||
| Target Milestone: | --- | ||
| Target Release: | --- | ||
| Hardware: | i386 | ||
| OS: | Linux | ||
| URL: | http://rt.freeciv.org/SelfService/Display.html?id=39898 | ||
| Whiteboard: | |||
| Fixed In Version: | 2.1.1-1.fc8 | Doc Type: | Bug Fix |
| Doc Text: | Story Points: | --- | |
| Clone Of: | Environment: | ||
| Last Closed: | 2007-12-07 21:33:58 UTC | Type: | --- |
| Regression: | --- | Mount Type: | --- |
| Documentation: | --- | CRM: | |
| Verified Versions: | Category: | --- | |
| oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
| Cloudforms Team: | --- | Target Upstream Version: | |
| Embargoed: | |||
|
Description
Jerry James
2007-11-24 00:49:41 UTC
I see the problem. The part array, declared at line 3342, has size PART_SIZE + 1. The variable bytes_adjust, declared at line 3339, is set to bytes_at_colon % 3; i.e., its value is 0, 1, or 2. The variable size_of_current_part, declared at line 3363, is PART_SIZE + bytes_adjust; i.e., it is at most PART_SIZE + 2. But then the offending memcpy copies size_of_current_part bytes into part, possibly overflowing it by one byte. The fix is to declare part as having size PART_SIZE + 2. I have now filed this bug upstream (#39898), and supplied them with a patch. See the bug URL. freeciv-2.1.0-2.fc8 has been pushed to the Fedora 8 testing repository. If problems still persist, please make note of it in this bug report. If you want to test the update, you can install it with su -c 'yum --enablerepo=updates-testing update freeciv' freeciv-2.1.1-1.fc8 has been pushed to the Fedora 8 testing repository. If problems still persist, please make note of it in this bug report. If you want to test the update, you can install it with su -c 'yum --enablerepo=updates-testing update freeciv' freeciv-2.1.1-1.fc8 has been pushed to the Fedora 8 stable repository. If problems still persist, please make note of it in this bug report. |