Note: There are a few cases of duplicates in user autocompletion which are being worked on.

bugzilla_email_append.pl calls processmail incorrectly

RESOLVED FIXED in Bugzilla 2.18

Status

()

Bugzilla
Bugzilla-General
P1
critical
RESOLVED FIXED
15 years ago
5 years ago

People

(Reporter: bbaetz, Assigned: bbaetz)

Tracking

unspecified
Bugzilla 2.18
x86
Linux

Details

(Whiteboard: [fixed in 2.16.1] [fixed in 2.14.4] [fixed on trunk])

Attachments

(1 attachment)

(Assignee)

Description

15 years ago
 
(Assignee)

Comment 1

15 years ago
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....
Severity: normal → critical
Priority: -- → P1
Whiteboard: [want for 2.16.1]
Target Milestone: --- → Bugzilla 2.18
(Assignee)

Comment 2

15 years ago
Created attachment 95666 [details] [diff] [review]
patch

This patch WFM. (modulo the usual bug_email bugs)
What happened to "cd .."?

Gerv
(Assignee)

Comment 4

15 years ago
Its not needed since bug 154008 went in - ie this is another breakage from that.
Assignee: justdave → bbaetz
Keywords: patch, review

Comment 5

15 years ago
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.
Attachment #95666 - Flags: review+

Comment 6

15 years ago
Comment on attachment 95666 [details] [diff] [review]
patch

People still use this stuff?!

Wild.

r2=preed
Attachment #95666 - Flags: review+
(Assignee)

Comment 7

15 years ago
Yes, people still use this (unfortunately)

Checked in to trunk and 2.16/2.14 branches
Status: NEW → RESOLVED
Last Resolved: 15 years ago
Resolution: --- → FIXED
Whiteboard: [want for 2.16.1] → [fixed in 2.16.1] [fixed in 2.14.4] [fixed on trunk]

Comment 8

15 years ago
Removing security bit for publication in status report.
Group: webtools-security?
(Assignee)

Comment 9

15 years ago
Err... Weren't we waiting for 2.16.1 before removing the sceuirity bit? Readding
the security bit for now....
Group: webtools-security?
Security Announcement is posted.  removing the security bit.
Group: webtools-security?
QA Contact: matty_is_a_geek → default-qa
You need to log in before you can comment on or make changes to this bug.