From: lpsolit%gmail.com <>
Date: Wed, 4 May 2005 01:32:58 +0000 (+0000)
Subject: Bug 287136: Missing validations in editproducts.cgi - Patch by Frédéric Buclin <LpSol... 
X-Git-Tag: bugzilla-2.19.3~16
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=3f138672bdd7fd4aba0b5c78b3541138174bd9f0;p=thirdparty%2Fbugzilla.git

Bug 287136: Missing validations in editproducts.cgi - Patch by Frédéric Buclin <LpSolit@gmail.com> r=joel a=justdave
---

diff --git a/editproducts.cgi b/editproducts.cgi
index be0842912e..3a63add6a1 100755
--- a/editproducts.cgi
+++ b/editproducts.cgi
@@ -425,6 +425,12 @@ if ($action eq 'new') {
 
     # Cleanups and validity checks
 
+    my $classification_id = 1;
+    if (Param('useclassification')) {
+        CheckClassification($classification);
+        $classification_id = get_classification_id($classification);
+    }
+
     unless ($product) {
         print "You must enter a name for the new product. Please press\n";
         print "<b>Back</b> and try again.\n";
@@ -475,11 +481,6 @@ if ($action eq 'new') {
     $votestoconfirm ||= 0;
     my $defaultmilestone = $cgi->param('defaultmilestone') || "---";
 
-    my $classification_id = 1;
-    if (Param('useclassification')) {
-        $classification_id = get_classification_id($classification);
-    }
-
     # Add the new product.
     SendSQL("INSERT INTO products ( " .
             "name, description, milestoneurl, disallownew, votesperuser, " .