markmcconnell / mai

Mark's AES67 Implementation
BSD 3-Clause "New" or "Revised" License
95 stars 22 forks source link

Unable to compile #1

Closed monoxane closed 5 years ago

monoxane commented 5 years ago

My error is

gcc -Wall -Werror -Wextra -std=c11 -O2 -fomit-frame-pointer -march=native   -c -o audio.o audio.c
gcc -Wall -Werror -Wextra -std=c11 -O2 -fomit-frame-pointer -march=native   -c -o jack.o jack.c
gcc -Wall -Werror -Wextra -std=c11 -O2 -fomit-frame-pointer -march=native   -c -o mai.o mai.c
gcc -Wall -Werror -Wextra -std=c11 -O2 -fomit-frame-pointer -march=native   -c -o ptp.o ptp.c
gcc -Wall -Werror -Wextra -std=c11 -O2 -fomit-frame-pointer -march=native   -c -o rtp.o rtp.c
gcc -Wall -Werror -Wextra -std=c11 -O2 -fomit-frame-pointer -march=native   -c -o sap.o sap.c
sap.c: In function 'sap':
sap.c:57:13: error: 'ptime' may be used uninitialized in this function [-Werror=maybe-uninitialized]
  payload += sprintf(payload, "a=ptime:%s\r\n", ptime);
             ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
cc1: all warnings being treated as errors
<builtin>: recipe for target 'sap.o' failed
make: *** [sap.o] Error 1

Using

gcc -v                                                                                                                                                          master
Using built-in specs.
COLLECT_GCC=gcc
COLLECT_LTO_WRAPPER=/usr/lib/gcc/x86_64-linux-gnu/6/lto-wrapper
Target: x86_64-linux-gnu
Configured with: ../src/configure -v --with-pkgversion='Debian 6.3.0-18+deb9u1' --with-bugurl=file:///usr/share/doc/gcc-6/README.Bugs --enable-languages=c,ada,c++,java,go,d,fortran,objc,obj-c++ --prefix=/usr --program-suffix=-6 --program-prefix=x86_64-linux-gnu- --enable-shared --enable-linker-build-id --libexecdir=/usr/lib --without-included-gettext --enable-threads=posix --libdir=/usr/lib --enable-nls --with-sysroot=/ --enable-clocale=gnu --enable-libstdcxx-debug --enable-libstdcxx-time=yes --with-default-libstdcxx-abi=new --enable-gnu-unique-object --disable-vtable-verify --enable-libmpx --enable-plugin --enable-default-pie --with-system-zlib --disable-browser-plugin --enable-java-awt=gtk --enable-gtk-cairo --with-java-home=/usr/lib/jvm/java-1.5.0-gcj-6-amd64/jre --enable-java-home --with-jvm-root-dir=/usr/lib/jvm/java-1.5.0-gcj-6-amd64 --with-jvm-jar-dir=/usr/lib/jvm-exports/java-1.5.0-gcj-6-amd64 --with-arch-directory=amd64 --with-ecj-jar=/usr/share/java/eclipse-ecj.jar --with-target-system-zlib --enable-objc-gc=auto --enable-multiarch --with-arch-32=i686 --with-abi=m64 --with-multilib-list=m32,m64,mx32 --enable-multilib --with-tune=generic --enable-checking=release --build=x86_64-linux-gnu --host=x86_64-linux-gnu --target=x86_64-linux-gnu
Thread model: posix

On Debian 9, have installed the deps listed in readme.md

rshamsnejad commented 5 years ago

This is solved by initializing the ptime pointer in sap.c, ie. modifying line 38 from

char *ptime;

to

char *ptime = NULL;

monoxane commented 5 years ago

This works

This is solved by initializing the ptime pointer in sap.c, ie. modifying line 38 from

char *ptime;

to

char *ptime = NULL;