Source: mpg321
Version: 0.3.2-3.1
Severity: serious
Tags: ftbfs
Justification: fails to build from source (but built successfully in the past)
X-Debbugs-Cc: sramac...@debian.org

https://buildd.debian.org/status/fetch.php?pkg=mpg321&arch=armel&ver=0.3.2-3.1%2Bb1&stamp=1710256758&raw=0

gcc -DHAVE_CONFIG_H -I.     -g -O2 -Werror=implicit-function-declaration 
-ffile-prefix-map=/<<PKGBUILDDIR>>=. -fstack-protector-strong 
-fstack-clash-protection -Wformat -Werror=format-security -Wall -Wunused 
-Wno-error=format-security -fcommon  -I/usr/include -DHAVE_ALSA -MT mpg321.o 
-MD -MP -MF .deps/mpg321.Tpo -c -o mpg321.o mpg321.c
mpg321.c:102:8: warning: type defaults to ‘int’ in declaration of 
‘http_file_length’ [-Wimplicit-int]
  102 | extern http_file_length;
      |        ^~~~~~~~~~~~~~~~
mpg321.c: In function ‘read_keyb’:
mpg321.c:132:27: error: implicit declaration of function 
‘mpg321_alsa_get_volume’ [-Werror=implicit-function-declaration]
  132 |                 bvolume = mpg321_alsa_get_volume();
      |                           ^~~~~~~~~~~~~~~~~~~~~~
mpg321.c:173:33: error: implicit declaration of function 
‘mpg321_alsa_set_volume’ [-Werror=implicit-function-declaration]
  173 |                                 mpg321_alsa_set_volume(bvolume);
      |                                 ^~~~~~~~~~~~~~~~~~~~~~
mpg321.c:186:75: warning: format ‘%ld’ expects argument of type ‘long int’, but 
argument 3 has type ‘int’ [-Wformat=]
  186 |                                                 fprintf(stderr,"Volume: 
%ld%%      \r",volume);
      |                                                                         
~~^            ~~~~~~
      |                                                                         
  |            |
      |                                                                         
  long int     int
      |                                                                         
%d
mpg321.c:236:75: warning: format ‘%ld’ expects argument of type ‘long int’, but 
argument 3 has type ‘int’ [-Wformat=]
  236 |                                                 fprintf(stderr,"Volume: 
%ld%%      \r",volume);
      |                                                                         
~~^            ~~~~~~
      |                                                                         
  |            |
      |                                                                         
  long int     int
      |                                                                         
%d
mpg321.c: In function ‘show_id3’:
mpg321.c:450:24: warning: format not a string literal and no format arguments 
[-Wformat-security]
  450 |                 printf(emptystring);
      |                        ^~~~~~~~~~~
mpg321.c:468:34: warning: format not a string literal and no format arguments 
[-Wformat-security]
  468 |                 fprintf (stderr, emptystring);
      |                                  ^~~~~~~~~~~
mpg321.c: In function ‘main’:
mpg321.c:688:17: error: implicit declaration of function 
‘init_alsa_volume_control’ [-Werror=implicit-function-declaration]
  688 |                 init_alsa_volume_control("default"); /* For the moment 
use "default", it works on most of the systems. Tested in 
Debian,Fedora,Ubuntu,RedHat,CentOS,Gentoo */
      |                 ^~~~~~~~~~~~~~~~~~~~~~~~
mpg321.c:882:13: error: implicit declaration of function ‘calc_http_length’; 
did you mean ‘calc_length’? [-Werror=implicit-function-declaration]
  882 |             calc_http_length(&playbuf);
      |             ^~~~~~~~~~~~~~~~
      |             calc_length
mpg321.c: In function ‘id3_get_tag’:
mpg321.c:1237:31: warning: pointer targets in passing argument 1 of ‘strlen’ 
differ in signedness [-Wpointer-sign]
 1237 |         if (!latin1 || strlen(latin1) == 0) return (NULL);
      |                               ^~~~~~
      |                               |
      |                               id3_latin1_t * {aka unsigned char *}
In file included from mpg321.c:35:
/usr/include/string.h:407:35: note: expected ‘const char *’ but argument is of 
type ‘id3_latin1_t *’ {aka ‘unsigned char *’}
  407 | extern size_t strlen (const char *__s)
      |                       ~~~~~~~~~~~~^~~
mpg321.c:1238:22: warning: pointer targets in passing argument 1 of ‘strlen’ 
differ in signedness [-Wpointer-sign]
 1238 |         len = strlen(latin1);
      |                      ^~~~~~
      |                      |
      |                      id3_latin1_t * {aka unsigned char *}
/usr/include/string.h:407:35: note: expected ‘const char *’ but argument is of 
type ‘id3_latin1_t *’ {aka ‘unsigned char *’}
  407 | extern size_t strlen (const char *__s)
      |                       ~~~~~~~~~~~~^~~
mpg321.c:1245:29: warning: pointer targets in passing argument 2 of ‘strncat’ 
differ in signedness [-Wpointer-sign]
 1245 |         strncat (printable, latin1, tocopy);
      |                             ^~~~~~
      |                             |
      |                             id3_latin1_t * {aka unsigned char *}
/usr/include/string.h:152:71: note: expected ‘const char * restrict’ but 
argument is of type ‘id3_latin1_t *’ {aka ‘unsigned char *’}
  152 | extern char *strncat (char *__restrict __dest, const char *__restrict 
__src,
      |                                                
~~~~~~~~~~~~~~~~~~~~~~~^~~~~
mpg321.c:1263:26: warning: pointer targets in passing argument 1 of ‘strlen’ 
differ in signedness [-Wpointer-sign]
 1263 |             len = strlen(latin1);
      |                          ^~~~~~
      |                          |
      |                          id3_latin1_t * {aka unsigned char *}
/usr/include/string.h:407:35: note: expected ‘const char *’ but argument is of 
type ‘id3_latin1_t *’ {aka ‘unsigned char *’}
  407 | extern size_t strlen (const char *__s)
      |                       ~~~~~~~~~~~~^~~
mpg321.c:1270:33: warning: pointer targets in passing argument 2 of ‘strncat’ 
differ in signedness [-Wpointer-sign]
 1270 |             strncat (printable, latin1, tocopy);
      |                                 ^~~~~~
      |                                 |
      |                                 id3_latin1_t * {aka unsigned char *}
/usr/include/string.h:152:71: note: expected ‘const char * restrict’ but 
argument is of type ‘id3_latin1_t *’ {aka ‘unsigned char *’}
  152 | extern char *strncat (char *__restrict __dest, const char *__restrict 
__src,
      |                                                
~~~~~~~~~~~~~~~~~~~~~~~^~~~~
mpg321.c:1270:13: warning: ‘strncat’ specified bound depends on the length of 
the source argument [-Wstringop-overflow=]
 1270 |             strncat (printable, latin1, tocopy);
      |             ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
mpg321.c:1263:19: note: length computed here
 1263 |             len = strlen(latin1);
      |                   ^~~~~~~~~~~~~~
mpg321.c:1245:9: warning: ‘strncat’ specified bound depends on the length of 
the source argument [-Wstringop-overflow=]
 1245 |         strncat (printable, latin1, tocopy);
      |         ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
mpg321.c:1238:15: note: length computed here
 1238 |         len = strlen(latin1);
      |               ^~~~~~~~~~~~~~
mpg321.c: In function ‘main’:
mpg321.c:1192:16: warning: ‘shm_id’ may be used uninitialized 
[-Wmaybe-uninitialized]
 1192 |             if(shmctl(shm_id,IPC_RMID,NULL))
      |                ^~~~~~~~~~~~~~~~~~~~~~~~~~~~
mpg321.c:521:9: note: ‘shm_id’ was declared here
  521 |     int shm_id;
      |         ^~~~~~
mpg321.c:1194:16: warning: ‘frames_id’ may be used uninitialized 
[-Wmaybe-uninitialized]
 1194 |             if(shmctl(frames_id,IPC_RMID,NULL))
      |                ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
mpg321.c:522:9: note: ‘frames_id’ was declared here
  522 |     int frames_id;
      |         ^~~~~~~~~
cc1: some warnings being treated as errors
make[2]: *** [Makefile:369: mpg321.o] Error 1

Cheers
-- 
Sebastian Ramacher

Reply via email to