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

_______________________________________________
pld-cvs-commit mailing list
[email protected]
http://lists.pld-linux.org/mailman/listinfo/pld-cvs-commit

Reply via email to