On 07/18/2013 02:01 PM, Denys Vlasenko wrote:
get_script_name() returns NULL if "python
/path/to/something"
and /path/to/something isn't packaged, or __if it is blacklisted__.
But then the caller, SavePackageDescriptionToDebugDump,
returns 1 as intended... only if (!settings_bProcessUnpackaged).
This is wrong if /path/to/something is blacklisted -
in this case, it should be unconditional.
This change moves "if (is_path_blacklisted(executable))"
check out of get_script_name() into the caller.
While as it, eliminate double checking of !settings_bProcessUnpackaged
by also moving
log("Interpreter crashed, but no packaged script detected: '%s'",
cmdline);
tot he caller.
Signed-off-by: Denys Vlasenko <dvlasenk(a)redhat.com>
The test for this bug had been committed to git:
abrt/tests/runtests/blacklisted-script