diff options
| author | Mark Sapiro <mark@msapiro.net> | 2009-12-30 13:39:08 -0800 |
|---|---|---|
| committer | Mark Sapiro <mark@msapiro.net> | 2009-12-30 13:39:08 -0800 |
| commit | 034ffbb9dc7dbee5c7e9ed73ebf346a04252b4bd (patch) | |
| tree | f53fed0c07a7130990527b8b5d0acd34bea9344e /bin/check_perms | |
| parent | 30ed3639227be2b174be06d69a43ba73cd205422 (diff) | |
| download | mailman2-034ffbb9dc7dbee5c7e9ed73ebf346a04252b4bd.tar.gz mailman2-034ffbb9dc7dbee5c7e9ed73ebf346a04252b4bd.tar.xz mailman2-034ffbb9dc7dbee5c7e9ed73ebf346a04252b4bd.zip | |
Fixed a bug where check_perms would throw an OSError if an entry in
Mailman's lists/ directory was not a directory. Bug #265613.
Diffstat (limited to '')
| -rwxr-xr-x | bin/check_perms | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/bin/check_perms b/bin/check_perms index 3c8d3768..137ebfb9 100755 --- a/bin/check_perms +++ b/bin/check_perms @@ -1,6 +1,6 @@ #! @PYTHON@ # -# Copyright (C) 1998-2008 by the Free Software Foundation, Inc. +# Copyright (C) 1998-2009 by the Free Software Foundation, Inc. # # This program is free software; you can redistribute it and/or # modify it under the terms of the GNU General Public License @@ -340,6 +340,8 @@ def checkdata(): print _('checking permissions on list data') # BAW: This needs to be converted to the Site module abstraction for dir in os.listdir(mm_cfg.LIST_DATA_DIR): + if not os.path.isdir(os.path.join(mm_cfg.LIST_DATA_DIR, dir)): + continue for file in checkfiles: path = os.path.join(mm_cfg.LIST_DATA_DIR, dir, file) if STATE.VERBOSE: |