chiark / gitweb /
fstab-generator: avoid mangling of mount source string
authorKay Sievers <kay@vrfy.org>
Mon, 4 Jun 2012 13:21:05 +0000 (15:21 +0200)
committerKay Sievers <kay@vrfy.org>
Mon, 4 Jun 2012 13:21:05 +0000 (15:21 +0200)
This is a valid source entry in /etc/fstab:
  //192.168.6.10/data /data cifs noperm,auto

On Mon, Jun 4, 2012 at 2:04 PM, Dave Reisner <d@falconindy.com> wrote:
> On Mon, Jun 04, 2012 at 12:57:47PM +0200, Kay Sievers wrote:
>>
>> Changed it to use:
>>   path_is_absolute()
>> instead of:
>>   is_path(),
>> so that we still sanitize the input we might match against.
>>
>> Let me know, if you think that could still cause any problems?

> Yes, this will still break CIFS shares.

src/fstab-generator/fstab-generator.c

index 0746724..d9ba3e3 100644 (file)
@@ -470,10 +470,7 @@ static int parse_fstab(void) {
                         goto finish;
                 }
 
-                if (path_is_absolute(what))
-                        path_kill_slashes(what);
-
-                if (path_is_absolute(where))
+                if (is_path(where))
                         path_kill_slashes(where);
 
                 log_debug("Found entry what=%s where=%s type=%s", what, where, me->mnt_type);