Fix FTBFS with gcc-16 and unused variables

https://bugs.debian.org/1133458

(Closes: #1133458)

Signed-off-by: Gianfranco Costamagna <locutusofborg@debian.org>
Signed-off-by: Gianfranco Costamagna <costamagnagianfranco@yahoo.it>

Author: LocutusOfBorg

doc/dlt_filetransfer.md

@@ -54,7 +54,7 @@ FLST | Package flag
 After the header package was sent, at least one or more data packages can be
 sent using:
 
-` int dlt_user_log_file_data(DltContext *fileContext,const char *filename,int packageToTransfer, int timeout) `
+` int dlt_user_log_file_data(DltContext *fileContext,const char *filename,int packageToTransfer, unsigned int timeout) `
 
 Data Data Package Protocol:
 

include/dlt/dlt_common.h

@@ -2308,7 +2308,7 @@ int dlt_execute_command(char *filename, char *command, ...);
  * @param filename Only file names without prepended path allowed.
  * @return pointer to extension
  */
-char *get_filename_ext(const char *filename);
+const char *get_filename_ext(const char *filename);
 
 /**
  * Extract the base name of given file name (without the extension).

include/dlt/dlt_filetransfer.h

@@ -70,7 +70,7 @@
  * @param timeout Timeout in ms to wait between some logs. Important that the FIFO of dlt will not be flooded with to many messages in a short period of time.
  * @return Returns 0 if everything was okey. If there was a failure value < 0 will be returned.
  */
-extern int dlt_user_log_file_complete(DltContext *fileContext, const char *filename, int deleteFlag, int timeout);
+extern int dlt_user_log_file_complete(DltContext *fileContext, const char *filename, int deleteFlag, unsigned int timeout);
 
 
 /* !This method gives information about the number of packages the file have */
@@ -126,7 +126,7 @@ extern int dlt_user_log_file_header(DltContext *fileContext, const char *filenam
  * @param fileCancelTransferFlag is a bool pointer to cancel the filetransfer on demand. For example in case of application shutdown event outstanding file transfer should abort and return
  * @return Returns 0 if everything was okey. If there was a failure value < 0 will be returned.
  */
-extern int dlt_user_log_file_data_cancelable(DltContext *fileContext, const char *filename, int packageToTransfer, int timeout, bool *const fileCancelTransferFlag);
+extern int dlt_user_log_file_data_cancelable(DltContext *fileContext, const char *filename, int packageToTransfer, unsigned int timeout, bool *const fileCancelTransferFlag);
 
 
 /* !Transfer the content data of a file. */
@@ -137,7 +137,7 @@ extern int dlt_user_log_file_data_cancelable(DltContext *fileContext, const char
  * @param timeout Timeout to wait between dlt logs. Important because the dlt FIFO should not be flooded. Default is defined by MIN_TIMEOUT. The given timeout in ms can not be smaller than MIN_TIMEOUT.
  * @return Returns 0 if everything was okey. If there was a failure value < 0 will be returned.
  */
-extern int dlt_user_log_file_data(DltContext *fileContext, const char *filename, int packageToTransfer, int timeout);
+extern int dlt_user_log_file_data(DltContext *fileContext, const char *filename, int packageToTransfer, unsigned int timeout);
 
 
 

src/console/dlt-control-common.c

@@ -349,7 +349,7 @@ static int prepare_headers(DltMessage *msg, uint8_t *header)
         return -1;
     }
 
-    msg->standardheader->len = DLT_HTOBE_16(len);
+    msg->standardheader->len = DLT_HTOBE_16((uint16_t)len);
 
     return 0;
 }

src/console/dlt-control-v2.c

@@ -787,8 +787,8 @@ int main(int argc, char *argv[])
         /*dlt_client_main_loop(&dltclient, &dltdata, dltdata.vflag); */
 
         /* Wait timeout */
-        ts.tv_sec = (dltdata.tvalue * NANOSEC_PER_MILLISEC) / NANOSEC_PER_SEC;
-        ts.tv_nsec = (dltdata.tvalue * NANOSEC_PER_MILLISEC) % NANOSEC_PER_SEC;
+        ts.tv_sec = (long int)(dltdata.tvalue * NANOSEC_PER_MILLISEC) / NANOSEC_PER_SEC;
+        ts.tv_nsec = (long int)(dltdata.tvalue * NANOSEC_PER_MILLISEC) % NANOSEC_PER_SEC;
         nanosleep(&ts, NULL);
     } else {
         ret = -1;
@@ -934,4 +934,4 @@ int dlt_receive_message_callback_v2(DltMessageV2 *message, void *data)
     }
 
     return ret;
-}
+}

src/console/dlt-control.c

@@ -754,8 +754,8 @@ int main(int argc, char *argv[])
         /*dlt_client_main_loop(&dltclient, &dltdata, dltdata.vflag); */
 
         /* Wait timeout */
-        ts.tv_sec = (dltdata.tvalue * NANOSEC_PER_MILLISEC) / NANOSEC_PER_SEC;
-        ts.tv_nsec = (dltdata.tvalue * NANOSEC_PER_MILLISEC) % NANOSEC_PER_SEC;
+        ts.tv_sec = (long int)(dltdata.tvalue * NANOSEC_PER_MILLISEC) / NANOSEC_PER_SEC;
+        ts.tv_nsec = (long int)(dltdata.tvalue * NANOSEC_PER_MILLISEC) % NANOSEC_PER_SEC;
         nanosleep(&ts, NULL);
     } else {
         ret = -1;

src/console/logstorage/dlt-logstorage-ctrl.c

@@ -545,7 +545,7 @@ static int parse_args(int argc, char *argv[])
 }
 
 #if !defined(DLT_SYSTEMD_ENABLE)
-int sd_notify(int unset_environment, const char *state)
+int sd_notify(int unset_environment, char *state)
 {
     /* Satisfy Compiler for warnings */
     (void)unset_environment;

src/console/logstorage/dlt-logstorage-udev.c

@@ -247,7 +247,7 @@ static int logstorage_udev_udevd_callback(void)
          * and/or for hot unplug (without unmount).
          */
         ts.tv_sec = 0;
-        ts.tv_nsec = 500 * NANOSEC_PER_MILLISEC;
+        ts.tv_nsec = (long int) 500 * NANOSEC_PER_MILLISEC;
         nanosleep(&ts, NULL);
         ret = check_mountpoint_from_partition(EVENT_MOUNTED, partition);
     }

src/daemon/dlt-daemon.c

@@ -1604,15 +1604,15 @@ int main(int argc, char *argv[])
     {
         char *watchdogUSec = getenv("WATCHDOG_USEC");
         // set a sensible default, in case the environment variable is not set
-        int watchdogTimeoutSeconds = 30;
+        unsigned int watchdogTimeoutSeconds = 30;
 
         dlt_log(LOG_DEBUG, "Systemd watchdog initialization\n");
 
         if (watchdogUSec) {
             // WATCHDOG_USEC is the timeout in micrsoseconds
             // divide this by 2*10^6 to get the interval in seconds
             // 2 * because we notify systemd after half the timeout
-            watchdogTimeoutSeconds = atoi(watchdogUSec) / 2000000;
+            watchdogTimeoutSeconds = (unsigned int)atoi(watchdogUSec) / 2000000;
         }
 
         if (watchdogTimeoutSeconds == 0) {
@@ -2874,7 +2874,7 @@ int dlt_daemon_log_internal(DltDaemon *daemon, DltDaemonLocal *daemon_local,
         msg.datasize += (int32_t)uiSize;
 
         /* Calc length */
-        msg.standardheader->len = DLT_HTOBE_16((size_t)msg.headersize - sizeof(DltStorageHeader) + (size_t)msg.datasize);
+        msg.standardheader->len = DLT_HTOBE_16((uint16_t)((size_t)msg.headersize - sizeof(DltStorageHeader) + (size_t)msg.datasize));
 
         dlt_daemon_client_send(DLT_DAEMON_SEND_TO_ALL, daemon,daemon_local,
                             msg.headerbuffer, sizeof(DltStorageHeader),
@@ -5660,8 +5660,8 @@ static void *timer_thread(void *data)
 #endif
 
 int create_timer_fd(DltDaemonLocal *daemon_local,
-                    int period_sec,
-                    int starts_in,
+                    unsigned int period_sec,
+                    unsigned int starts_in,
                     DltTimers timer_id)
 {
     int local_fd = DLT_FD_INIT;
@@ -5679,7 +5679,7 @@ int create_timer_fd(DltDaemonLocal *daemon_local,
         return -1;
     }
 
-    if ((period_sec <= 0) || (starts_in <= 0)) {
+    if ((period_sec == 0) || (starts_in == 0)) {
         /* timer not activated via the service file */
         dlt_vlog(LOG_INFO, "<%s> not set: period=0\n", timer_name);
         local_fd = DLT_FD_INIT;
@@ -5693,10 +5693,10 @@ int create_timer_fd(DltDaemonLocal *daemon_local,
             dlt_vlog(LOG_WARNING, "<%s> timerfd_create failed: %s\n",
                      timer_name, strerror(errno));
 
-        l_timer_spec.it_interval.tv_sec = period_sec;
-        l_timer_spec.it_interval.tv_nsec = 0;
-        l_timer_spec.it_value.tv_sec = starts_in;
-        l_timer_spec.it_value.tv_nsec = 0;
+        l_timer_spec.it_interval.tv_sec = (long int) period_sec;
+        l_timer_spec.it_interval.tv_nsec = (long int) 0;
+        l_timer_spec.it_value.tv_sec = (long int) starts_in;
+        l_timer_spec.it_value.tv_nsec = (long int) 0;
 
         if (timerfd_settime(local_fd, 0, &l_timer_spec, NULL) < 0) {
             dlt_vlog(LOG_WARNING, "<%s> timerfd_settime failed: %s\n",

src/daemon/dlt-daemon.h

@@ -190,8 +190,8 @@ typedef struct
 typedef struct
 {
     unsigned long long wakeups_missed;
-    int period_sec;
-    int starts_in;
+    unsigned int period_sec;
+    unsigned int starts_in;
     int timer_id;
 } DltDaemonPeriodicData;
 
@@ -232,7 +232,7 @@ int dlt_daemon_process_client_messages_serial(DltDaemon *daemon,
 int dlt_daemon_process_user_messages(DltDaemon *daemon, DltDaemonLocal *daemon_local, DltReceiver *recv, int verbose);
 int dlt_daemon_process_one_s_timer(DltDaemon *daemon, DltDaemonLocal *daemon_local, DltReceiver *recv, int verbose);
 int dlt_daemon_process_sixty_s_timer(DltDaemon *daemon, DltDaemonLocal *daemon_local, DltReceiver *recv, int verbose);
-int dlt_daemon_process_systemd_timer(DltDaemon *daemon, DltDaemonLocal *daemon_local, DltReceiver *recv, int verbose);
+ssize_t dlt_daemon_process_systemd_timer(DltDaemon *daemon, DltDaemonLocal *daemon_local, DltReceiver *recv, int verbose);
 
 int dlt_daemon_process_control_connect(DltDaemon *daemon, DltDaemonLocal *daemon_local, DltReceiver *recv, int verbose);
 #if defined DLT_DAEMON_USE_UNIX_SOCKET_IPC || defined DLT_DAEMON_VSOCK_IPC_ENABLE
@@ -300,7 +300,7 @@ void dlt_daemon_ecu_version_thread(void *ptr);
 void dlt_daemon_systemd_watchdog_thread(void *ptr);
 #endif
 
-int create_timer_fd(DltDaemonLocal *daemon_local, int period_sec, int starts_in, DltTimers timer);
+int create_timer_fd(DltDaemonLocal *daemon_local, unsigned int period_sec, unsigned int starts_in, DltTimers timer);
 
 int dlt_daemon_close_socket(int sock, DltDaemon *daemon, DltDaemonLocal *daemon_local, int verbose);