0
0
mirror of https://github.com/openwrt/packages.git synced 2025-10-13 18:45:28 +00:00
Files
packages/net/atftp/patches/0002-Fix-pthread_t-format-warning-for-fprintf.patch
Florian Eckert 62dfcf026f atftp: add patch to fix build warning
This patch fixes the following compilation build warning:

logger.c:117:47: warning: format '%li' expects argument of type 'long int', but argument 7 has type
  117 |                fprintf(log_fp, "%s %s %s[%d.%li]: %s\n", time_buf, hostname,
      |                                             ~~^
      |                                               |
      |                                               long int
  118 |                        log_ident, getpid(), pthread_self(), message);
      |                                             ~~~~~~~~~~~~~~
      |                                             |
      |                                             pthread_t {aka struct __pthread *}
logger.c:124:47: warning: format '%li' expects argument of type 'long int', but argument 7 has type
  124 |                fprintf(stderr, "%s %s %s[%d.%li]: %s\n", time_buf, hostname,
      |                                             ~~^
      |                                               |
      |                                               long int
  125 |                        log_ident, getpid(), pthread_self(), message);
      |                                             ~~~~~~~~~~~~~~
      |                                             |
      |                                             pthread_t {aka struct __pthread *}

I have also submitted the change to the upstream project:
https://github.com/madmartin/atftp/pull/2

Signed-off-by: Florian Eckert <fe@dev.tdt.de>
2025-09-08 09:20:43 +02:00

51 lines
2.5 KiB
Diff

From a9da732718bd917a50aa25a6ae59e791c2c30380 Mon Sep 17 00:00:00 2001
From: Florian Eckert <fe@dev.tdt.de>
Date: Tue, 2 Sep 2025 13:53:05 +0200
Subject: [PATCH] Fix pthread_t format warning for fprintf
This change fixes the following compilation build warning:
logger.c:117:47: warning: format '%li' expects argument of type 'long int', but argument 7 has type 'pthread_t' {aka 'struct __pthread *'} [-Wformat=]
117 | fprintf(log_fp, "%s %s %s[%d.%li]: %s\n", time_buf, hostname,
| ~~^
| |
| long int
118 | log_ident, getpid(), pthread_self(), message);
| ~~~~~~~~~~~~~~
| |
| pthread_t {aka struct __pthread *}
logger.c:124:47: warning: format '%li' expects argument of type 'long int', but argument 7 has type 'pthread_t' {aka 'struct __pthread *'} [-Wformat=]
124 | fprintf(stderr, "%s %s %s[%d.%li]: %s\n", time_buf, hostname,
| ~~^
| |
| long int
125 | log_ident, getpid(), pthread_self(), message);
| ~~~~~~~~~~~~~~
| |
| pthread_t {aka struct __pthread *}
Signed-off-by: Florian Eckert <fe@dev.tdt.de>
---
logger.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
--- a/logger.c
+++ b/logger.c
@@ -115,14 +115,14 @@ void logger(int severity, const char *fm
if (log_fp)
{
fprintf(log_fp, "%s %s %s[%d.%li]: %s\n", time_buf, hostname,
- log_ident, getpid(), pthread_self(), message);
+ log_ident, getpid(), (unsigned long int) pthread_self(), message);
fflush(log_fp);
}
else if (log_syslog_is_open)
syslog(severity, "%s", message);
else
fprintf(stderr, "%s %s %s[%d.%li]: %s\n", time_buf, hostname,
- log_ident, getpid(), pthread_self(), message);
+ log_ident, getpid(), (unsigned long int) pthread_self(), message);
}
va_end(args);
}