RE: [patch][2.5.4-dj4] cleanup to use strsep for fs/fat/inode.c

• Next message: Alan Cox: "Re: APM fix from -pre7 seems to break "Dell laptop support""
• Previous message: Marc Recht: "Re: funny console prob w/2.4.18-pre[479]+kpreempt+sched-o(1)"
• In reply to: René Scharfe: "Re: [patch][2.5.4-dj4] cleanup to use strsep for fs/fat/inode.c"
• Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

> > This patch changes all use of strtok() to strsep().
> > Strtok() isn't SMP/thread safe. strsep is considered safer.
>
> OK, but ...
>
> > - for (this_char = strtok(options,","); this_char;
> > - this_char = strtok(NULL,",")) {
>
> This _does not_ change the value of 'options'.
>
> > + while ((this_char = strsep(&options,",")) != NULL) {
>
> This _does_ change the value of 'options'. Problem is, it's
> used later. Same is true for your patch to fs/vfat/namei.c.

Ok, sorry seems i overlooked that. Patches, for both files, below
should fix it.

Cheers,

--
	Alex (alex@packetstorm.nu)
diff -uN linux-2.5.3-dj4/fs/vfat/namei.c linux/fs/vfat/namei.c
--- linux-2.5.3-dj4/fs/vfat/namei.c     Mon Jan 28 22:20:44 2002
+++ linux/fs/vfat/namei.c       Sun Feb 10 15:28:26 2002
@@ -97,6 +97,7 @@
 static int parse_options(char *options,        struct fat_mount_options *opts)
 {
        char *this_char,*value,save,*savep;
+       char *optptr = options;
        int ret, val;
        opts->unicode_xlate = opts->posixfs = 0;
@@ -114,7 +115,9 @@
        save = 0;
        savep = NULL;
        ret = 1;
-       for (this_char = strtok(options,","); this_char; this_char = strtok(NULL,",")) {
+       while ((this_char = strsep(&optptr,",")) != NULL) {
+               if (!*this_char)
+                       continue;
                if ((value = strchr(this_char,'=')) != NULL) {
                        save = *value;
                        savep = value;
diff -uN linux-2.5.3-dj4/fs/fat/inode.c linux/fs/fat/inode.c
--- linux-2.5.3-dj4/fs/fat/inode.c      Sat Feb  9 21:57:39 2002
+++ linux/fs/fat/inode.c        Sun Feb 10 15:51:42 2002
@@ -203,7 +203,7 @@
                         char *cvf_format, char *cvf_options)
 {
        char *this_char,*value,save,*savep;
-       char *p;
+       char *p, *optptr = options;
        int ret = 1, len;
        opts->name_check = 'n';
@@ -223,8 +223,9 @@
                goto out;
        save = 0;
        savep = NULL;
-       for (this_char = strtok(options,","); this_char;
-            this_char = strtok(NULL,",")) {
+       while ((this_char = strsep(&optptr,",")) != NULL) {
+               if (!*this_char)
+                       continue;
                if ((value = strchr(this_char,'=')) != NULL) {
                        save = *value;
                        savep = value;
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

• Next message: Alan Cox: "Re: APM fix from -pre7 seems to break "Dell laptop support""
• Previous message: Marc Recht: "Re: funny console prob w/2.4.18-pre[479]+kpreempt+sched-o(1)"
• In reply to: René Scharfe: "Re: [patch][2.5.4-dj4] cleanup to use strsep for fs/fat/inode.c"
• Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

This archive was generated by hypermail 2b29 : Fri Feb 15 2002 - 21:00:32 EST