#85 mount.davfs processes '-o conf=' incorrectly

closed-fixed
5
2008-07-28
2008-06-27
No

The case statement that process the '-o' argument doesn't work correctly for the 'conf' long option; it resets the mount UID as a side-effect. This affects the CVS version (1.3.2).

Here is a fix:

Index: davfs2-1.3.2/src/mount_davfs.c

--- davfs2-1.3.2.orig/src/mount_davfs.c
+++ davfs2-1.3.2/src/mount_davfs.c
@@ -1519,6 +1519,7 @@ static void get_options(dav_args *args,
if (args->conf != NULL)
free(args->conf);
args->conf = ne_strdup(argument);
+ break;
case UID:
pwd = getpwnam(argument);
if (pwd == NULL) {

Discussion

  • Werner Baumann

    Werner Baumann - 2008-06-30
    • assigned_to: nobody --> wbaumann
    • status: open --> open-fixed
     
  • Werner Baumann

    Werner Baumann - 2008-06-30

    Logged In: YES
    user_id=1260327
    Originator: NO

    Thanks for the report and the bugfix. I added it to the CVS-version.

    Cheers
    Werner

     
  • Werner Baumann

    Werner Baumann - 2008-07-28
    • status: open-fixed --> closed-fixed
     

Log in to post a comment.