237591 - Regexp metachars in product names break importxml.pl

Status: RESOLVED FIXED

(Bugzilla :: Bug Import/Export & Moving, defect)

Description

[Richard Giraud]

User-Agent:       Opera/7.23 (Windows NT 5.0; U)  [en]
Build Identifier: 

importxml.pl does not properly import bugs.  Instead, it gives the following 
message:

Bug import error: invalid default product or component

Reproducible: Always
Steps to Reproduce:
1. Create XML representation of a bug via xml.cgi
2. Save to disk.
3. Run importxml.pl on file (e.g., sudo perl importxml.pl bug.xml).

Actual Results:  
Import stopped during first bug with following message.

Bug import error: invalid default product or component

Expected Results:  
Imported the bug.

Around line 380 or so, importxml.pl filters the arrays of products and 
components using the grep command.  An examination reveals that grep filters out 
all elements in the array.

The product name is correctly determined a few lines earlier and a comparison of 
the product name to the first element of the @::legal_product array returns a 
value of 1.

Similar problems occur with the component checking.

The version of PERL being used is v5.8.1-RC3.

I will include example code shortly (it is on a different box).

Comment 1

[Dave Miller [:justdave]]

What version of Bugzilla?

There is no "grep" in use in the current version leading up to the mentioned
error message.  It's just calling get_product_id() on it and giving the error if
that fails.

Comment 2

[Richard Giraud]

Attachment: Console log from execution

Comment 3

[Richard Giraud]

Attachment: The XML definition of the bug

This is the minimal dataset that I have tested that causes the problem.

Comment 4

[Dave Miller [:justdave]]

OK, found the code in question in 2.16.5.

Someone wrote some really sloppy grep code there.  No quoting of the variables,
and your product names have regexp metachars in them.

Adding \Q and \E around the variable names will probably fix it, but doing
{ lc($_) eq lc($product_name) } in place of the regexp would probably be a much
better fix.

Comment 5

[Richard Giraud]

Attachment: importxml.pl file with added print statements for debugging

I have added some print statements to display the internal state of the script
at the appropriate points.

Comment 6

[Dave Miller [:justdave]]

Attachment: Patch v1

Give this patch a shot and let me know if this fixes it.

Comment 7

[Richard Giraud]

That solved the problem.  Thank you very much :)

Comment 8

[Dave Miller [:justdave]]

Checking in importxml.pl;
/cvsroot/mozilla/webtools/bugzilla/importxml.pl,v  <--  importxml.pl
new revision: 1.23.2.3; previous revision: 1.23.2.2
done

Comment 9

[Marc Schumann [:Wurblzap]]

Comment on attachment 143999 [details] [diff] [review]
Patch v1

Removing r? from FIXED bug.