Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 275200 - media-gfx/jhead-2.8.7 version bump + respect cflags + respect ldflags + mktemp
Summary: media-gfx/jhead-2.8.7 version bump + respect cflags + respect ldflags + mktemp
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: All Linux
: High normal
Assignee: Marcelo Goes (RETIRED)
URL: http://www.sentex.net/~mwandel/jhead/
Whiteboard:
Keywords:
Depends on:
Blocks: ldflags
  Show dependency tree
 
Reported: 2009-06-23 14:48 UTC by Zeev Tarantov
Modified: 2010-08-09 21:00 UTC (History)
1 user (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
patch from ebuild in portage to new ebuild (jhead_ebuild.patch,824 bytes, patch)
2009-06-23 14:51 UTC, Zeev Tarantov
Details | Diff
patch to makefile to respect flags (jhead-respect_flags.patch,502 bytes, patch)
2009-06-23 14:51 UTC, Zeev Tarantov
Details | Diff
patch to change use of mktemp to mkstemp (jhead-mkstemp.patch,383 bytes, patch)
2009-06-23 14:52 UTC, Zeev Tarantov
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Zeev Tarantov 2009-06-23 14:48:33 UTC
There is a new version of jhead, although the changelog is confused about version number.

The ebuild does not respect CFLAGS and LDFLAGS.

Also, there is a warning about using mktemp(3) instead of mkstemp(3).

I'm attaching two patches and a patch to the ebuild.

Reproducible: Always




I am not a security person, but I don't think the mstemp thing is an exploitable security vulnerability, that's why I'm not reporting this as a security bug.
Comment 1 Zeev Tarantov 2009-06-23 14:51:13 UTC
Created attachment 195561 [details, diff]
patch from ebuild in portage to new ebuild
Comment 2 Zeev Tarantov 2009-06-23 14:51:36 UTC
Created attachment 195562 [details, diff]
patch to makefile to respect flags
Comment 3 Zeev Tarantov 2009-06-23 14:52:22 UTC
Created attachment 195564 [details, diff]
patch to change use of mktemp to mkstemp
Comment 4 Marcelo Goes (RETIRED) gentoo-dev 2009-06-28 17:35:57 UTC
Thanks Zeev, now in cvs.
Please, be sure to send the mkstemp patch to upstream.

Cheers,
Marcelo