extras-buildsys/common FileDownloader.py,1.13,1.14

[Daniel Williams (dcbw)]

Author: dcbw

Update of /cvs/fedora/extras-buildsys/common
In directory cvs-int.fedora.redhat.com:/tmp/cvs-serv27510/common

Modified Files:
	FileDownloader.py 
Log Message:
2005-07-26  Dan Williams <dcbw at redhat.com>

    - Throttle CVS checkouts to 5 jobs at a time
    - Validate package and cvs_tag inputs to enqueue functions since they
        get passed directly to commands.getstatusoutput()




Index: FileDownloader.py
===================================================================
RCS file: /cvs/fedora/extras-buildsys/common/FileDownloader.py,v
retrieving revision 1.13
retrieving revision 1.14
diff -u -r1.13 -r1.14
--- FileDownloader.py	21 Jul 2005 14:36:49 -0000	1.13
+++ FileDownloader.py	26 Jul 2005 17:47:19 -0000	1.14
@@ -70,9 +70,10 @@
         raise FileNameError("Extension was not allowed.")
 
     # FIXME: what other validation can we do here?
+    safe_list = ['_', '-', '.', '+']
     for c in filename:
         # For now, legal characters are '_-.' plus alphanumeric
-        if (c == '_') or (c == '-') or (c == '.') or (c == '+') or c.isalnum():
+        if c in safe_list or c.isalnum():
             pass
         else:
             raise FileNameError("Illegal character '%s' encountered." % c)