Bug 1037409
| Summary: | z88dk FTBFS if "-Werror=format-security" flag is used | ||||||||
|---|---|---|---|---|---|---|---|---|---|
| Product: | [Fedora] Fedora | Reporter: | Dhiru Kholia <dkholia> | ||||||
| Component: | z88dk | Assignee: | Kevin Kofler <kevin> | ||||||
| Status: | CLOSED RAWHIDE | QA Contact: | Fedora Extras Quality Assurance <extras-qa> | ||||||
| Severity: | unspecified | Docs Contact: | |||||||
| Priority: | unspecified | ||||||||
| Version: | rawhide | CC: | bressers, dennis, dhiru, kevin | ||||||
| Target Milestone: | --- | ||||||||
| Target Release: | --- | ||||||||
| Hardware: | Unspecified | ||||||||
| OS: | Unspecified | ||||||||
| Whiteboard: | |||||||||
| Fixed In Version: | z88dk-1.10.1-8.20150709cvs.fc23 | Doc Type: | Bug Fix | ||||||
| Doc Text: | Story Points: | --- | |||||||
| Clone Of: | Environment: | ||||||||
| Last Closed: | 2015-07-09 21:45:05 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: | |||||||||
| Bug Depends On: | |||||||||
| Bug Blocks: | 1038083, 1105908 | ||||||||
| Attachments: |
|
||||||||
|
Description
Dhiru Kholia
2013-12-03 06:08:38 UTC
Created attachment 834008 [details]
FTBFS fix
Created attachment 834009 [details]
fixed .spec file
Building this packages results in lot of warnings. Please coordinate with upstream to fix the code. Even perfectly valid code isn't good / secure / useful by itself. Thank you for the patch, but using fprintf with a "%s" format string is silly, that's what fputs is for. :-) That's what I'm going to use. This is already fixed in upstream CVS (using fputs): http://z88dk.cvs.sourceforge.net/viewvc/z88dk/z88dk/src/z80asm/die.c?hideattic=0&r1=1.9&r2=1.10 And in the latest upstram code, the functions simply look like this: http://z88dk.cvs.sourceforge.net/viewvc/z88dk/z88dk/src/z80asm/lib/die.c?revision=1.3&view=markup Upstream also fixed their warnings. I need to upgrade to current CVS anyway because of bug #967408, so I will probably just do that. (In reply to Kevin Kofler from comment #4) > Thank you for the patch, but using fprintf with a "%s" format string is > silly, that's what fputs is for. :-) That's what I'm going to use. heh ;) Sounds good, thanks! *** Bug 1106700 has been marked as a duplicate of this bug. *** This bug appears to have been reported against 'rawhide' during the Fedora 22 development cycle. Changing version to '22'. More information and reason for this action is here: https://fedoraproject.org/wiki/Fedora_Program_Management/HouseKeeping/Fedora22 Fixed by upgrading to a new upstream snapshot. |