Skip to content

Commit

Permalink
[modesetting] Fix message/command formatting issues
Browse files Browse the repository at this point in the history
Avoid literal buffer sizes, remove unnecessary trailing whitespace,
and use correct data types.

Signed-off-by: Simo Piiroinen <simo.piiroinen@jollamobile.com>
  • Loading branch information
spiiroin committed Jan 28, 2020
1 parent 5942e6c commit 3a950e8
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions src/usb_moded-modesetting.c
Expand Up @@ -526,7 +526,7 @@ static bool modesetting_enter_mass_storage_mode(const modedata_t *data)
{
log_debug("%s does not exist, unloading and reloading mass_storage\n", tmp);
modules_unload_module(MODULE_MASS_STORAGE);
snprintf(tmp, sizeof tmp, "modprobe %s luns=%zd \n", MODULE_MASS_STORAGE, count);
snprintf(tmp, sizeof tmp, "modprobe %s luns=%zd", MODULE_MASS_STORAGE, count);
log_debug("usb-load command = %s \n", tmp);
if( common_system(tmp) != 0 )
goto EXIT;
Expand Down Expand Up @@ -724,7 +724,7 @@ bool modesetting_enter_dynamic_mode(void)
}

log_debug("data->mass_storage = %d", data->mass_storage);
log_debug("data->connman_tethering = %d", data->connman_tethering);
log_debug("data->connman_tethering = %s", data->connman_tethering ?: "n/a");
log_debug("data->appsync = %d", data->appsync);
log_debug("data->network = %d", data->network);

Expand Down Expand Up @@ -800,7 +800,7 @@ bool modesetting_enter_dynamic_mode(void)
#ifdef DEBIAN
char command[256];

g_snprintf(command, 256, "ifdown %s ; ifup %s", data->network_interface, data->network_interface);
g_snprintf(command, sizeof command, "ifdown %s ; ifup %s", data->network_interface, data->network_interface);
common_system(command);
#else
network_down(data);
Expand Down Expand Up @@ -873,7 +873,7 @@ void modesetting_leave_dynamic_mode(void)
}

log_debug("data->mass_storage = %d", data->mass_storage);
log_debug("data->connman_tethering = %d", data->connman_tethering);
log_debug("data->connman_tethering = %s", data->connman_tethering ?: "n/a");
log_debug("data->appsync = %d", data->appsync);
log_debug("data->network = %d", data->network);

Expand Down

0 comments on commit 3a950e8

Please sign in to comment.