On Mon, Dec 26, 2011 at 06:47:24PM -0500, Dave Reisner wrote:
> In line with previous patches, the source of a pseudofs is strictly
> semantic; do not attempt to resolve it to a filesystem path.
>
> As a side effect of this, nspec needs to be initialized to NULL since
> it may not be assigned null by spec_to_devname when the FS is deemed
> a pseudofs.
>
> Signed-off-by: Dave Reisner <dreisner@xxxxxxxxxxxxx>
> ---
> This wasn't as evil as I thought, after all...
>
> mount/mount.c | 5 +++--
> 1 files changed, 3 insertions(+), 2 deletions(-)
>
> diff --git a/mount/mount.c b/mount/mount.c
> index 9b444c4..49d49f6 100644
> --- a/mount/mount.c
> +++ b/mount/mount.c
> @@ -2016,7 +2016,7 @@ is_existing_file (const char *s) {
> static int
> mount_one (const char *spec, const char *node, const char *types,
> const char *fstabopts, char *cmdlineopts, int freq, int pass) {
> - const char *nspec;
> + const char *nspec = NULL;
> char *opts;
>
> /* Substitute values in opts, if required */
> @@ -2046,7 +2046,8 @@ mount_one (const char *spec, const char *node, const char *types,
> strncmp(types, "nfs", 3) &&
> strncmp(types, "cifs", 4) &&
> strncmp(types, "smbfs", 5))) {
> - nspec = spec_to_devname(spec);
> + if (!is_pseudo_fs(types))
> + nspec = spec_to_devname(spec);
Exactly this is implemented in spec_to_devname(), it does not
canonicalize pseudo filesystems.
> if (nspec)
> spec = nspec;
> }
> --
> 1.7.8.1
>
> --
> To unsubscribe from this list: send the line "unsubscribe util-linux" in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> More majordomo info at http://vger.kernel.org/majordomo-info.html
>
--
Karel Zak <kzak@xxxxxxxxxx>
http://karelzak.blogspot.com
--
To unsubscribe from this list: send the line "unsubscribe util-linux" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
