mirrors/neutrino
1
0
Fork 0
mirror of https://github.com/tuxbox-neutrino/neutrino.git synced 2025-08-31 17:31:20 +02:00
Code Issues Packages Projects Releases Wiki Activity

hdd_menu: fix -Wformat-truncation warnings it feels more sane to avoid possible truncation instead of just disarming the gcc test

Browse Source 
Signed-off-by: Markus Volk <f_l_k@t-online.de>
This commit is contained in:
Markus Volk
2020-02-01 08:47:19 +01:00
parent b0dee3c9f3
commit 04b426412e
1 changed files with 11 additions and 11 deletions
Download Patch File Download Diff File Expand all files Collapse all files

22
src/gui/hdd_menu.cpp
View File

@@ -664,7 +664,7 @@ bool CHDDMenuHandler::scanDevices()
printf("HDD: root_dev: 0x%04x\n", root_dev); printf("HDD: root_dev: 0x%04x\n", root_dev);
for(int i = 0; i < n;i++) { for(int i = 0; i < n;i++) {
char str[256]; char str[281];
char vendor[128] = { 0 }; char vendor[128] = { 0 };
char model[128] = { 0 }; char model[128] = { 0 };
int64_t bytes = 0; int64_t bytes = 0;
@@ -673,7 +673,7 @@ bool CHDDMenuHandler::scanDevices()
bool isroot = false; bool isroot = false;
printf("HDD: checking /sys/block/%s\n", namelist[i]->d_name); printf("HDD: checking /sys/block/%s\n", namelist[i]->d_name);
snprintf(str, sizeof(str), "/dev/%s", namelist[i]->d_name) < 0 ? abort() : (void)0; snprintf(str, sizeof(str), "/dev/%s", namelist[i]->d_name);
int fd = open(str, O_RDONLY); int fd = open(str, O_RDONLY);
if (fd >= 0) { if (fd >= 0) {
if (ioctl(fd, BLKGETSIZE64, &bytes)) if (ioctl(fd, BLKGETSIZE64, &bytes))
@@ -696,11 +696,11 @@ bool CHDDMenuHandler::scanDevices()
megabytes = bytes/1000000; megabytes = bytes/1000000;
snprintf(str, sizeof(str), "/sys/block/%s/device/vendor", namelist[i]->d_name) < 0 ? abort() : (void)0; snprintf(str, sizeof(str), "/sys/block/%s/device/vendor", namelist[i]->d_name);
FILE * f = fopen(str, "r"); FILE * f = fopen(str, "r");
if(!f) { if(!f) {
printf("Cant open %s\n", str); printf("Cant open %s\n", str);
snprintf(str, sizeof(str), "/sys/block/%s/device/type", namelist[i]->d_name) < 0 ? abort() : (void)0; snprintf(str, sizeof(str), "/sys/block/%s/device/type", namelist[i]->d_name);
f = fopen(str, "r"); f = fopen(str, "r");
} }
if (f) { if (f) {
@@ -713,13 +713,13 @@ bool CHDDMenuHandler::scanDevices()
/* the Tripledragon only has kernel 2.6.12 available.... :-( */ /* the Tripledragon only has kernel 2.6.12 available.... :-( */
if (oldkernel) if (oldkernel)
snprintf(str, sizeof(str), "/proc/ide/%s/model", namelist[i]->d_name) < 0 ? abort() : (void)0; snprintf(str, sizeof(str), "/proc/ide/%s/model", namelist[i]->d_name);
else else
snprintf(str, sizeof(str), "/sys/block/%s/device/model", namelist[i]->d_name) < 0 ? abort() : (void)0; snprintf(str, sizeof(str), "/sys/block/%s/device/model", namelist[i]->d_name);
f = fopen(str, "r"); f = fopen(str, "r");
if(!f) { if(!f) {
printf("Cant open %s\n", str); printf("Cant open %s\n", str);
snprintf(str, sizeof(str), "/sys/block/%s/device/name", namelist[i]->d_name) < 0 ? abort() : (void)0; snprintf(str, sizeof(str), "/sys/block/%s/device/name", namelist[i]->d_name);
f = fopen(str, "r"); f = fopen(str, "r");
} }
if (f) { if (f) {
@@ -728,7 +728,7 @@ bool CHDDMenuHandler::scanDevices()
} }
#if 0 #if 0
int removable = 0; int removable = 0;
snprintf(str, sizeof(str), "/sys/block/%s/removable", namelist[i]->d_name) < 0 ? abort() : (void)0; snprintf(str, sizeof(str), "/sys/block/%s/removable", namelist[i]->d_name);
f = fopen(str, "r"); f = fopen(str, "r");
if(!f) { if(!f) {
printf("Cant open %s\n", str); printf("Cant open %s\n", str);
@@ -750,7 +750,7 @@ bool CHDDMenuHandler::scanDevices()
hdd_list.push_back(hdd); hdd_list.push_back(hdd);
} }
snprintf(str, sizeof(str), "%s %s %ld %s", vendor, model, (long)(megabytes < 10000 ? megabytes : megabytes/1000), megabytes < 10000 ? "MB" : "GB") < 0 ? abort() : (void)0; snprintf(str, sizeof(str), "%s %s %ld %s", vendor, model, (long)(megabytes < 10000 ? megabytes : megabytes/1000), megabytes < 10000 ? "MB" : "GB");
printf("HDD: %s\n", str); printf("HDD: %s\n", str);
devtitle[namelist[i]->d_name] = str; devtitle[namelist[i]->d_name] = str;
@@ -1417,10 +1417,10 @@ int CHDDDestExec::exec(CMenuTarget* /*parent*/, const std::string&)
printf("CHDDDestExec: noise %d sleep %d /dev/%s\n", printf("CHDDDestExec: noise %d sleep %d /dev/%s\n",
g_settings.hdd_noise, g_settings.hdd_sleep, namelist[i]->d_name); g_settings.hdd_noise, g_settings.hdd_sleep, namelist[i]->d_name);
char M_opt[50],S_opt[50], opt[100]; char M_opt[50],S_opt[50], opt[261];
snprintf(S_opt, sizeof(S_opt), "-S%d", g_settings.hdd_sleep); snprintf(S_opt, sizeof(S_opt), "-S%d", g_settings.hdd_sleep);
snprintf(M_opt, sizeof(M_opt), "-M%d", g_settings.hdd_noise); snprintf(M_opt, sizeof(M_opt), "-M%d", g_settings.hdd_noise);
snprintf(opt, sizeof(opt), "/dev/%s",namelist[i]->d_name) < 0 ? abort() : (void)0; snprintf(opt, sizeof(opt), "/dev/%s",namelist[i]->d_name);
if (have_nonbb_hdparm) if (have_nonbb_hdparm)
my_system(4, hdparm, M_opt, S_opt, opt); my_system(4, hdparm, M_opt, S_opt, opt);