mirrors/recycled-ni-neutrino
1
0
Fork 0
mirror of https://github.com/tuxbox-fork-migrations/recycled-ni-neutrino.git synced 2025-09-02 18:31:12 +02:00
Code Issues Packages Projects Releases Wiki Activity

hdd_menu: use realpath(3) instead of calling system("readlink")

Browse Source 
Origin commit data
------------------
Commit: f8066b5a3e
Author: Stefan Seyfried <seife@tuxbox-git.slipkontur.de>
Date: 2015-02-10 (Tue, 10 Feb 2015)
This commit is contained in:
Stefan Seyfried
2015-02-10 14:11:20 +01:00
parent e47735fe44
commit 2dbff76c62
1 changed files with 11 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

14
src/gui/hdd_menu.cpp
View File

@@ -133,6 +133,14 @@ int CHDDMenuHandler::filterDevName(const char * name)
return 0;
}
static string readlink(const char *path)
{
char link[PATH_MAX + 1];
if (realpath(path, link))
return string(link);
return "";
}
bool CHDDMenuHandler::is_mounted(const char *dev)
{
bool res = false;
@@ -143,7 +151,7 @@ bool CHDDMenuHandler::is_mounted(const char *dev)
snprintf(devpath, sizeof(devpath), "/dev/%s", dev);
char buffer[255];
string realdev = backtick("readlink -f " + string(devpath));
string realdev = readlink(devpath);
realdev = trim(realdev);
FILE *f = fopen("/proc/mounts", "r");
if(f) {
@@ -156,7 +164,7 @@ bool CHDDMenuHandler::is_mounted(const char *dev)
if (!strcmp(buffer, devpath)) /* default '/dev/sda1' mount */
res = true;
else { /* now the case of '/dev/disk/by-label/myharddrive' mounts */
string realmount = backtick("readlink -f " + string(buffer));
string realmount = readlink(buffer);
if (realdev == trim(realmount))
res = true;
}
@@ -410,7 +418,7 @@ void CHDDMenuHandler::setRecordPath(std::string &dev)
return;
}
/* don't annoy if the recordingdir is a symlink pointing to the 'right' location */
string readl = backtick("readlink -f " + g_settings.network_nfs_recordingdir);
string readl = readlink(g_settings.network_nfs_recordingdir.c_str());
readl = trim(readl);
if (newpath.compare(readl) == 0) {
printf("CHDDMenuHandler::%s: recordingdir is a symlink to %s\n",