PLD-doc: PLDSA/new-sa - vulnerability typo (maybe choose shorter v...

[glen]

Author: glen                         Date: Sun Jun  4 19:06:57 2006 GMT
Module: PLD-doc                       Tag: HEAD
---- Log message:
- vulnerability typo (maybe choose shorter variable name?)
- more checks

---- Files affected:
PLD-doc/PLDSA:
   new-sa (1.1 -> 1.2) 

---- Diffs:

================================================================
Index: PLD-doc/PLDSA/new-sa
diff -u PLD-doc/PLDSA/new-sa:1.1 PLD-doc/PLDSA/new-sa:1.2
--- PLD-doc/PLDSA/new-sa:1.1	Sun Jun  4 21:03:23 2006
+++ PLD-doc/PLDSA/new-sa	Sun Jun  4 21:06:52 2006
@@ -35,7 +35,7 @@
 sa_file=$db/$(printf %02d $id).pldsa
 sa_id="PLDSA $year-$(printf %02d $id)"
 
-vulnreability=
+vulnerability=
 problem_type="local/remote"
 pld_specific="yes/no"
 cve_id=
@@ -44,7 +44,7 @@
 
 # parse command line args
 t=`getopt -o p: --long \
-package:,cve-id:,bugtraq-id:,upstream-url:,vulnreability:,local,remote \
+package:,cve-id:,bugtraq-id:,upstream-url:,vulnerability:,local,remote \
 -n "$self" -- "$@"`
 if [ $? != 0 ] ; then echo "Terminating..." >&2 ; exit 1 ; fi
 eval set -- "$t"
@@ -67,9 +67,9 @@
 			shift
 			upstream_url="$1"
 		;;
-		--vulnreability)
+		--vulnerability)
 			shift
-			vulnreability="$1"
+			vulnerability="$1"
 		;;
 		--local)
 			problem_type=local
@@ -97,10 +97,10 @@
 	<br>
 	Package (Name-Version-Release) : <input id="package" caption="$package"><br>
 	<br>
-	Vulnreability is short description of the problem.<br>
+	Vulnerability is short description of the problem.<br>
 	It will be included in announce Subject.<br>
 	<br>
-	Vulnerability   : <input id="vulnreability" caption="$vulnreability"><br>
+	Vulnerability   : <input id="vulnerability" caption="$vulnerability"><br>
 	Problem-Type    : <input id="problem_type" caption="$problem_type"><br>
 	PLD specific    : <input id="pld_specific" caption="$pld_specific"><br>
 	<br>
@@ -135,8 +135,16 @@
 		echo "Aborting..."
 		exit 0
 	fi
+	if [ "$package" = "" ]; then
+		error "Package name is required field!"
+		continue
+	fi
 	if [ "$pld_specific" = "yes/no" ]; then
-		error "Field 'PLD specific' must be filled!"
+		error "PLD specific must be filled!"
+		continue
+	fi
+	if [ "$problem_type" = "local/remote" ]; then
+		error "Problem type (local or remote) must be filled!"
 		continue
 	fi
 	break
@@ -150,7 +158,7 @@
 ----------------------------------------------------------------------------
 
 Package       : $package
-Vulnerability : $vulnreability
+Vulnerability : $vulnerability
 Problem-Type  : $problem_type
 PLD-specific  : $pld_specific
 
================================================================

---- CVS-web:
    http://cvs.pld-linux.org/PLD-doc/PLDSA/new-sa?r1=1.1&r2=1.2&f=u