lkml.org 
[lkml]   [2002]   [Feb]   [10]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
From
SubjectRE: [patch][2.5.4-dj4] cleanup to use strsep for fs/fat/inode.c
Date
> > 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/

\
 
 \ /
  Last update: 2005-03-22 13:24    [W:0.038 / U:10.484 seconds]
©2003-2018 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site