Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 340251 - net-analyzer/hping _FORTIFY_SOURCE indicates presence of overflow
Summary: net-analyzer/hping _FORTIFY_SOURCE indicates presence of overflow
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: High major (vote)
Assignee: Netmon Herd
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: fortify-source
  Show dependency tree
 
Reported: 2010-10-09 12:15 UTC by Diego Elio Pettenò (RETIRED)
Modified: 2010-11-30 21:31 UTC (History)
2 users (show)

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


Attachments
Build log (hping-3_pre20051105-r2:20101009-093109.log,7.63 KB, text/plain)
2010-10-09 12:15 UTC, Diego Elio Pettenò (RETIRED)
Details
Patch to hping-3_pre20051105-r2.ebuild to address memcpy overflow (hping-3_pre20051105-r2.ebuild.patch,433 bytes, patch)
2010-10-09 18:29 UTC, Kevin Pyle
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Diego Elio Pettenò (RETIRED) gentoo-dev 2010-10-09 12:15:17 UTC
You're receiving this bug because the package in Summary has produced _FORTIFY_SOURCE related warnings indicating the presence of a sure overflow in a static buffer.

Even though this is not always an indication of a security problem it might even be. So please check this out ASAP.

By the way, _FORTIFY_SOURCE is disabled when you disable optimisation, so don't try finding out the cause using -O0.

Thanks,
Your friendly neighborhood tinderboxer
Comment 1 Diego Elio Pettenò (RETIRED) gentoo-dev 2010-10-09 12:15:38 UTC
Created attachment 250011 [details]
Build log
Comment 2 Kevin Pyle 2010-10-09 18:29:20 UTC
Created attachment 250043 [details, diff]
Patch to hping-3_pre20051105-r2.ebuild to address memcpy overflow

The value passed as sizeof() to the memcpy was unrelated to the value being filled by the memcpy, so the computed size was too big.  This patch adds a sed to rewrite the offending memcpy to use sizeof on the value receiving the data.
Comment 3 Jeroen Roovers gentoo-dev 2010-11-30 21:31:16 UTC
Fixed in hping-3_pre20051105-r3.ebuild.