163024 - bugzilla_email_append.pl calls processmail incorrectly

Status: RESOLVED FIXED

(Bugzilla :: Bugzilla-General, defect, P1)

Description

[Bradley Baetz (:bbaetz)]

Comment 1

[Bradley Baetz (:bbaetz)]

Bleh. Stupid enter key...

Anyway, as jussi pointed out in bug 160631, bugzilla_email_append.pl uses the
same quoting that bug_email.pl used, which was fixed for bug 154008:

system("cd .. ; ./processmail $found_id '$SenderShort'");

is not the way to go. We should fix that....

Comment 2

[Bradley Baetz (:bbaetz)]

Attachment: patch

This patch WFM. (modulo the usual bug_email bugs)

Comment 3

[Gervase Markham [:gerv]]

What happened to "cd .."?

Gerv

Comment 4

[Bradley Baetz (:bbaetz)]

Its not needed since bug 154008 went in - ie this is another breakage from that.

Comment 5

[Joel Peshkin]

Comment on attachment 95666 [details] [diff] [review]
patch


r=joel assuming that someone has confirmed that the routined still does its
desired function.
Still need a seperate 2xr for 2.16 branch.

Comment 6

[J. Paul Reed [:preed]]

Comment on attachment 95666 [details] [diff] [review]
patch

People still use this stuff?!

Wild.

r2=preed

Comment 7

[Bradley Baetz (:bbaetz)]

Yes, people still use this (unfortunately)

Checked in to trunk and 2.16/2.14 branches

Comment 8

[J. Paul Reed [:preed]]

Removing security bit for publication in status report.

Comment 9

[Bradley Baetz (:bbaetz)]

Err... Weren't we waiting for 2.16.1 before removing the sceuirity bit? Readding
the security bit for now....

Comment 10

[Dave Miller [:justdave]]

Security Announcement is posted.  removing the security bit.