X-Git-Url: https://git.decadent.org.uk/gitweb/?p=nfs-utils.git;a=blobdiff_plain;f=utils%2Fmountd%2Frmtab.c;fp=utils%2Fmountd%2Frmtab.c;h=0a4ed4c6b1b09269978c0462a0f67354f163b756;hp=170d8594e4371f7373f50b7ba78cfd6d620d3a8e;hb=e85f72300043f96e3f352d9c8f93243d4d0a52dc;hpb=0036a143c415eac6388d992a4e637e7004147296 diff --git a/utils/mountd/rmtab.c b/utils/mountd/rmtab.c index 170d859..0a4ed4c 100644 --- a/utils/mountd/rmtab.c +++ b/utils/mountd/rmtab.c @@ -8,7 +8,9 @@ #include "config.h" +#include #include +#include #include #include #include @@ -18,6 +20,40 @@ #include "xio.h" #include "mountd.h" +#include /* PATH_MAX */ + +/* If new path is a link do not destroy it but place the + * file where the link points. + */ + +static int +slink_safe_rename(const char * oldpath, const char * newpath) +{ + int r; + struct stat s; + char slink_path[PATH_MAX]; + char real_newpath = newpath; + + if((lstat(newpath, &s) == 0) && (S_ISLNK(s.st_mode))) { + + /* New path is a symbolic link, do not destroy but follow */ + + if((r = readlink(newpath, slink_path, PATH_MAX))==-1) { + + return -1; + + } + + slink_path[ (r < PATH_MAX) ? (r + 1) : (PATH_MAX - 1)] = '\0'; + + real_newpath = slink_path; + + } + + return rename(oldpath, real_newpath); + +}/* static int slink_safe_rename() */ + void mountlist_add(nfs_export *exp, const char *path) { @@ -82,7 +118,7 @@ mountlist_del(nfs_export *exp, const char *path) if (!match || rep->r_count) fputrmtabent(fp, rep, NULL); } - if (rename(_PATH_RMTABTMP, _PATH_RMTAB) < 0) { + if (slink_safe_rename(_PATH_RMTABTMP, _PATH_RMTAB) < 0) { xlog(L_ERROR, "couldn't rename %s to %s", _PATH_RMTABTMP, _PATH_RMTAB); } @@ -130,7 +166,7 @@ mountlist_del_all(struct sockaddr_in *sin) } fputrmtabent(fp, rep, NULL); } - if (rename(_PATH_RMTABTMP, _PATH_RMTAB) < 0) { + if (slink_safe_rename(_PATH_RMTABTMP, _PATH_RMTAB) < 0) { xlog(L_ERROR, "couldn't rename %s to %s", _PATH_RMTABTMP, _PATH_RMTAB); }