]> err.no Git - util-linux/commitdiff
tests: disable suid mount test
authorKarel Zak <kzak@redhat.com>
Wed, 18 Feb 2009 22:33:25 +0000 (23:33 +0100)
committerKarel Zak <kzak@redhat.com>
Wed, 18 Feb 2009 22:33:25 +0000 (23:33 +0100)
Unfortunately, libtool wrapper cannot be suid...

Signed-off-by: Karel Zak <kzak@redhat.com>
tests/ts/mount/fstab-symlink

index 98797a7f476efb6af4553421837f76572f81dca6..41e51b4a6aaace3eff95185edee437ecfb7b6c5f 100755 (executable)
@@ -36,6 +36,7 @@ ts_device_has "TYPE" "ext3" $DEVICE || ts_die "Cannot found ext3 on $DEVICE" $DE
 
 [ -d "$TS_MOUNTPOINT" ] || mkdir -p $TS_MOUNTPOINT
 
+rm -rf $LINKNAME
 ln -s $DEVICE $LINKNAME
 
 ts_fstab_add $LINKNAME $TS_MOUNTPOINT "auto" "defaults,user"
@@ -45,14 +46,18 @@ $TS_CMD_MOUNT $LINKNAME 2>&1 >> $TS_OUTPUT
 ts_is_mounted $DEVICE || ts_die "A) Cannot found $DEVICE in /proc/mounts" $DEVICE
 $TS_CMD_UMOUNT $LINKNAME || ts_die "A) Cannot umount $LINKNAME" $DEVICE
 
-# varian B)
-ts_init_suid $TS_CMD_MOUNT
-ts_init_suid $TS_CMD_UMOUNT
+# varian B) -- disabled, the $TS_CMD_MOUNT is usually not a real program, but
+#              libtool wrapper and the real mount is exec()ed with
+#              LD_LIBRARY_PATH. This all is bad for suid programs...
 
-su $TS_TESTUSER -c "$TS_CMD_MOUNT $LINKNAME" 2>&1 >> $TS_OUTPUT
-ts_is_mounted $DEVICE || ts_die "B) Cannot found $DEVICE in /proc/mounts" $DEVICE
-su $TS_TESTUSER -c "$TS_CMD_UMOUNT $LINKNAME" 2>&1 >> $TS_OUTPUT \
- || ts_die "B) Cannot umount $LINKNAME" $DEVICE
+#ts_init_suid $TS_CMD_MOUNT
+#ts_init_suid $TS_CMD_UMOUNT
+#
+#su $TS_TESTUSER -c "$TS_CMD_MOUNT -v -v -v $LINKNAME" 2>&1 >> $TS_OUTPUT
+#ts_is_mounted $DEVICE
+#   || ts_die "B) Cannot found $DEVICE in /proc/mounts" $DEVICE
+#su $TS_TESTUSER -c "$TS_CMD_UMOUNT $LINKNAME" 2>&1 >> $TS_OUTPUT \
+#   || ts_die "B) Cannot umount $LINKNAME" $DEVICE
 
 ts_device_deinit $DEVICE
 ts_fstab_clean