aboutsummaryrefslogtreecommitdiffstats
path: root/camel/providers
diff options
context:
space:
mode:
authorJeffrey Stedfast <fejj@ximian.com>2004-03-26 02:27:41 +0800
committerJeffrey Stedfast <fejj@src.gnome.org>2004-03-26 02:27:41 +0800
commitc8c4777d6d0928e7dd2f68aeff5a43e3e660c12a (patch)
tree65dd1f2ddc6385ffaeddc78039f5940646c42973 /camel/providers
parent41810f55a293ed42e3c1cfc19accad1f1fc174e9 (diff)
downloadgsoc2013-evolution-c8c4777d6d0928e7dd2f68aeff5a43e3e660c12a.tar
gsoc2013-evolution-c8c4777d6d0928e7dd2f68aeff5a43e3e660c12a.tar.gz
gsoc2013-evolution-c8c4777d6d0928e7dd2f68aeff5a43e3e660c12a.tar.bz2
gsoc2013-evolution-c8c4777d6d0928e7dd2f68aeff5a43e3e660c12a.tar.lz
gsoc2013-evolution-c8c4777d6d0928e7dd2f68aeff5a43e3e660c12a.tar.xz
gsoc2013-evolution-c8c4777d6d0928e7dd2f68aeff5a43e3e660c12a.tar.zst
gsoc2013-evolution-c8c4777d6d0928e7dd2f68aeff5a43e3e660c12a.zip
Fix for bug #55018.
2004-03-25 Jeffrey Stedfast <fejj@ximian.com> Fix for bug #55018. * providers/imap/camel-imap-store.c (create_folder): Don't allow the suer to create folders with #, %, * or the directory separator in the folder name (added the checks for %, * and #). (get_folder_online): Add a check to make sure the folder name is sane before sending a CREATE (ie. we want to allow getting of folders with discouraged characters in them if they exist, but we don't want to allow the user to create them). svn path=/trunk/; revision=25186
Diffstat (limited to 'camel/providers')
-rw-r--r--camel/providers/imap/camel-imap-store.c27
1 files changed, 23 insertions, 4 deletions
diff --git a/camel/providers/imap/camel-imap-store.c b/camel/providers/imap/camel-imap-store.c
index b9ced3e01b..f60b91c683 100644
--- a/camel/providers/imap/camel-imap-store.c
+++ b/camel/providers/imap/camel-imap-store.c
@@ -1703,7 +1703,8 @@ get_folder_online (CamelStore *store, const char *folder_name, guint32 flags, Ca
response = camel_imap_command (imap_store, NULL, ex, "SELECT %F", folder_name);
if (!response) {
char *folder_real;
-
+ const char *c;
+
if (camel_exception_get_id(ex) == CAMEL_EXCEPTION_USER_CANCEL) {
CAMEL_SERVICE_UNLOCK (imap_store, connect_lock);
return NULL;
@@ -1717,7 +1718,20 @@ get_folder_online (CamelStore *store, const char *folder_name, guint32 flags, Ca
_("No such folder %s"), folder_name);
return NULL;
}
-
+
+ c = folder_name;
+ while (*c && *c != imap_store->dir_sep && !strchr ("#%*", *c))
+ c++;
+
+ if (*c != '\0') {
+ CAMEL_SERVICE_UNLOCK (imap_store, connect_lock);
+ camel_exception_setv (ex, CAMEL_EXCEPTION_FOLDER_INVALID_PATH,
+ _("The folder name \"%s\" is invalid because "
+ "it containes the character \"%c\""),
+ folder_name, *c);
+ return NULL;
+ }
+
folder_real = camel_imap_store_summary_path_to_full(imap_store->summary, folder_name, store->dir_sep);
response = camel_imap_command (imap_store, NULL, ex, "CREATE %S", folder_real);
@@ -1988,17 +2002,22 @@ create_folder (CamelStore *store, const char *parent_name,
CamelFolderInfo *root = NULL;
gboolean need_convert;
int i = 0, flags;
+ const char *c;
if (!camel_disco_store_check_online (CAMEL_DISCO_STORE (store), ex))
return NULL;
if (!parent_name)
parent_name = "";
- if (strchr (folder_name, imap_store->dir_sep)) {
+ c = folder_name;
+ while (*c && *c != imap_store->dir_sep && !strchr ("#%*", *c))
+ c++;
+
+ if (*c != '\0') {
camel_exception_setv (ex, CAMEL_EXCEPTION_FOLDER_INVALID_PATH,
_("The folder name \"%s\" is invalid because "
"it containes the character \"%c\""),
- folder_name, imap_store->dir_sep);
+ folder_name, *c);
return NULL;
}