578208 – app-forensics/chkrootkit-0.50: bad variable in chk_dirname() function

Bug 578208 - app-forensics/chkrootkit-0.50: bad variable in chk_dirname() function

Summary: app-forensics/chkrootkit-0.50: bad variable in chk_dirname() function

Status:	RESOLVED TEST-REQUEST

Alias:	None

Product:	Gentoo Linux
Classification:	Unclassified
Component:	Current packages (show other bugs)
Hardware:	All Linux

Importance:	Normal normal (vote)
Assignee:	No maintainer - Look at https://wiki.gentoo.org/wiki/Project:Proxy_Maintainers if you want to take care of it

URL:
Whiteboard:
Keywords:

Depends on:
Blocks:

Reported:	2016-03-25 06:29 UTC by Adam Randall
Modified:	2019-04-26 02:47 UTC (History)
CC List:	1 user (show)

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

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.

Description Adam Randall 2016-03-25 06:29:04 UTC

In comparing the differences between version 0.49 and 0.50 of chkrootkit I noticed that there is an invalid variable being referenced on line 1825:

    if ${ls} -l ${MD} | ${egrep} "^...s" > /dev/null 2>&1
    then
        STATUS=${INFECTED}
    fi

It should instead be:

    if ${ls} -l ${CMD} | ${egrep} "^...s" > /dev/null 2>&1
    then
        STATUS=${INFECTED}
    fi

Note the use of ${MD} vs ${CMD}

This comparison is used several other places in the code correctly. To be double sure, I looked for any reference to a ${MD} variable, and other than line 1825, it does not exist.

Comment 1 Agostino Sarubbo gentoo-dev

2016-07-08 13:51:50 UTC

Hello.

Could you please make a Pull request and send it via github? it will be applied.

Thanks.

Comment 2 Pacho Ramos gentoo-dev

2017-08-30 18:28:10 UTC

Please retry with 0.51 version