Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 57701 - app-admin/chkrootkit: strings-static is not static
Summary: app-admin/chkrootkit: strings-static is not static
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All All
: High normal (vote)
Assignee: solar (RETIRED)
URL:
Whiteboard:
Keywords: InVCS
Depends on:
Blocks:
 
Reported: 2004-07-20 03:19 UTC by Aaron Walker (RETIRED)
Modified: 2004-07-28 21:06 UTC (History)
0 users

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


Attachments
03-chkrootkit-0.43-r3.diff (03-chkrootkit-0.43-r3.diff,490 bytes, patch)
2004-07-20 03:26 UTC, Aaron Walker (RETIRED)
Details | Diff
chkrootkit-0.43-r2.diff (chkrootkit-0.43-r2.diff,498 bytes, patch)
2004-07-24 14:03 UTC, Aaron Walker (RETIRED)
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Aaron Walker (RETIRED) gentoo-dev 2004-07-20 03:19:12 UTC
chkrootkit's strings-static binary is not statically compiled because of the line, "make strings || die" in the ebuild, since there is no "strings" target in the
Makefile (but strings.c exists, so it builds it anyways instead of complaining).

The fix is just to remove that line altogether, since strings-static gets built with the "sense" target.  This also means we can get rid of the call to newsbin and just add strings-static to dosbin.
Comment 1 Aaron Walker (RETIRED) gentoo-dev 2004-07-20 03:26:33 UTC
Created attachment 35801 [details, diff]
03-chkrootkit-0.43-r3.diff
Comment 2 Aaron Walker (RETIRED) gentoo-dev 2004-07-24 14:03:05 UTC
Created attachment 36091 [details, diff]
chkrootkit-0.43-r2.diff

The other patch was an incremental patch based on an ebuild patched to fix
another chkrootkit bug.  So, here is the diff of the current
chkrootkit-0.43-r2.ebuild
Comment 3 solar (RETIRED) gentoo-dev 2004-07-28 21:06:13 UTC
playing ping pong with me guys?

It must be destiny that this bug made it's way to me for a second time after
I've already bounced this back to bug-wranglers@ before.

Alot of chkrootkit internals were overridden by some patch (not your patch Aaron) which made it use everything dynamic. 
This is a flaw in the design on that patch as
the goal should be to use known safe virus free programs/and dynamic
linker and libraries to preform our tests. 

Consider a small LD_PRELOAD module which overrode these dynamic programs 
functions and hide the non trojaned values from us.

This strings-static might be just one of the programs which are effected
by this patch (not your patch Aaron).

Anyway I've put this in CVS as-is to help return some of the sainity for forensics.