[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#966847: wine-development: FTBFS: cpanelfolder.c:262:5: error: ‘strcpy’ offset 12 from the object at ‘pidl’ is out of the bounds of referenced subobject ‘szName’ with type ‘CHAR[1]’ {aka ‘char[1]’} at offset 12 [-Werror=array-bounds]



Source: wine-development
Version: 5.5-5
Severity: serious
Justification: FTBFS on amd64
Tags: bullseye sid ftbfs
Usertags: ftbfs-20200802 ftbfs-bullseye

Hi,

During a rebuild of all packages in sid, your package failed to build
on amd64.

Relevant part (hopefully):
> make[2]: Entering directory '/<<PKGBUILDDIR>>/dlls/sxs'
> x86_64-w64-mingw32-gcc -c -o cache.cross.o cache.c -I. -I../../include -I../../include/msvcrt -D__WINESRC__ -D_UCRT -DWINE_CROSS_PE -Wall -Wdate-time -g -O2 -fdebug-prefix-map=/<<PKGBUILDDIR>>=. -Wformat -Werror=format-security -Wno-shift-overflow -Wno-unused-function -Wno-deprecated-declarations -fno-strict-aliasing
> ../../tools/winegcc/winegcc -o sti.dll --wine-objdir ../.. -b x86_64-w64-mingw32 --lib-suffix=.cross.a -Wl,--wine-builtin -shared sti.spec -mno-cygwin sti.cross.o sti_main.cross.o dlldata.cross.o sti_wia_p.cross.o sti_wia_r.res ../../dlls/uuid/libuuid.cross.a ../../dlls/ole32/libole32.cross.a ../../dlls/oleaut32/liboleaut32.cross.a ../../dlls/rpcrt4/librpcrt4.cross.a ../../dlls/advapi32/libadvapi32.cross.a ../../dlls/ucrtbase/libucrtbase.cross.a 
> make[2]: Leaving directory '/<<PKGBUILDDIR>>/dlls/sti'
> x86_64-w64-mingw32-gcc -c -o name.cross.o name.c -I. -I../../include -I../../include/msvcrt -D__WINESRC__ -D_UCRT -DWINE_CROSS_PE -Wall -Wdate-time -g -O2 -fdebug-prefix-map=/<<PKGBUILDDIR>>=. -Wformat -Werror=format-security -Wno-shift-overflow -Wno-unused-function -Wno-deprecated-declarations -fno-strict-aliasing
> x86_64-w64-mingw32-gcc -c -o sxs.cross.o sxs.c -I. -I../../include -I../../include/msvcrt -D__WINESRC__ -D_UCRT -DWINE_CROSS_PE -Wall -Wdate-time -g -O2 -fdebug-prefix-map=/<<PKGBUILDDIR>>=. -Wformat -Werror=format-security -Wno-shift-overflow -Wno-unused-function -Wno-deprecated-declarations -fno-strict-aliasing
> In function ‘_ILCreateCPanelApplet’,
>     inlined from ‘SHELL_RegisterCPanelApp’ at cpanelfolder.c:323:20:
> cpanelfolder.c:262:5: error: ‘strcpy’ offset 12 from the object at ‘pidl’ is out of the bounds of referenced subobject ‘szName’ with type ‘CHAR[1]’ {aka ‘char[1]’} at offset 12 [-Werror=array-bounds]
>   262 |     strcpy(p->szName, name);
>       |     ^~~~~~~~~~~~~~~~~~~~~~~
> In file included from cpanelfolder.c:44:
> cpanelfolder.c: In function ‘SHELL_RegisterCPanelApp’:
> pidl.h:119:10: note: subobject ‘szName’ declared here
>   119 |     CHAR szName[1];  /*10*/ /* terminated by 0x00, followed by display name and comment string */
>       |          ^~~~~~
> In file included from sxs.c:27:
> sxs.c: In function ‘SxsLookupClrGuid’:
> sxs.c:114:11: warning: format ‘%lx’ expects argument of type ‘long unsigned int’, but argument 9 has type ‘SIZE_T’ {aka ‘long long unsigned int’} [-Wformat=]
>   114 |     TRACE("(%x, %s, %p, %p, %08lx, %p): stub\n", flags, wine_dbgstr_guid(clsid), actctx,
>       |           ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
>   115 |           buffer, buffer_len, buffer_len_required);
>       |                   ~~~~~~~~~~
>       |                   |
>       |                   SIZE_T {aka long long unsigned int}
> ../../include/wine/debug.h:89:49: note: in definition of macro ‘__WINE_DBG_LOG’
>    89 |     wine_dbg_log( __dbcl, __dbch, __FUNCTION__, args); } } while(0)
>       |                                                 ^~~~
> ../../include/wine/debug.h:472:36: note: in expansion of macro ‘__WINE_DPRINTF’
>   472 | #define WINE_TRACE                 __WINE_DPRINTF(_TRACE,__wine_dbch___default)
>       |                                    ^~~~~~~~~~~~~~
> ../../include/wine/debug.h:515:36: note: in expansion of macro ‘WINE_TRACE’
>   515 | #define TRACE                      WINE_TRACE
>       |                                    ^~~~~~~~~~
> sxs.c:114:5: note: in expansion of macro ‘TRACE’
>   114 |     TRACE("(%x, %s, %p, %p, %08lx, %p): stub\n", flags, wine_dbgstr_guid(clsid), actctx,
>       |     ^~~~~
> sxs.c:114:33: note: format string is defined here
>   114 |     TRACE("(%x, %s, %p, %p, %08lx, %p): stub\n", flags, wine_dbgstr_guid(clsid), actctx,
>       |                             ~~~~^
>       |                                 |
>       |                                 long unsigned int
>       |                             %08I64x
> make[2]: Entering directory '/<<PKGBUILDDIR>>/dlls/t2embed'
> x86_64-w64-mingw32-gcc -c -o main.cross.o main.c -I. -I../../include -I../../include/msvcrt -D__WINESRC__ -D_UCRT -DWINE_CROSS_PE -Wall -Wdate-time -g -O2 -fdebug-prefix-map=/<<PKGBUILDDIR>>=. -Wformat -Werror=format-security -Wno-shift-overflow -Wno-unused-function -Wno-deprecated-declarations -fno-strict-aliasing
> gcc -m64 -c -o dataobject.o dataobject.c -I. -I../../include -I../.. -D__WINESRC__ -D_SHELL32_ -D_REENTRANT -fPIC -Wall -pipe -fcf-protection=none -fno-stack-protector -fno-strict-aliasing -Wdeclaration-after-statement -Wempty-body -Wignored-qualifiers -Wno-packed-not-aligned -Wshift-overflow=2 -Wstrict-prototypes -Wtype-limits -Wunused-but-set-parameter -Wvla -Wwrite-strings -Wpointer-arith -Wlogical-op -gdwarf-2 -gstrict-dwarf -Werror -Wdate-time -g -O2 -fdebug-prefix-map=/<<PKGBUILDDIR>>=. -Wformat -Werror=format-security -Wno-shift-overflow -Wno-unused-function -Wno-deprecated-declarations
> ../../tools/winegcc/winegcc -o sxs.dll --wine-objdir ../.. -b x86_64-w64-mingw32 --lib-suffix=.cross.a -Wl,--wine-builtin -shared sxs.spec -mno-cygwin cache.cross.o name.cross.o sxs.cross.o ../../dlls/oleaut32/liboleaut32.cross.a ../../dlls/ole32/libole32.cross.a ../../dlls/ucrtbase/libucrtbase.cross.a 
> make[2]: Leaving directory '/<<PKGBUILDDIR>>/dlls/sxs'
> make[2]: Entering directory '/<<PKGBUILDDIR>>/dlls/tapi32'
> x86_64-w64-mingw32-gcc -c -o assisted.cross.o assisted.c -I. -I../../include -I../../include/msvcrt -D__WINESRC__ -D_UCRT -DWINE_CROSS_PE -Wall -Wdate-time -g -O2 -fdebug-prefix-map=/<<PKGBUILDDIR>>=. -Wformat -Werror=format-security -Wno-shift-overflow -Wno-unused-function -Wno-deprecated-declarations -fno-strict-aliasing
> ../../tools/winegcc/winegcc -o t2embed.dll --wine-objdir ../.. -b x86_64-w64-mingw32 --lib-suffix=.cross.a -Wl,--wine-builtin -shared t2embed.spec -mno-cygwin main.cross.o ../../dlls/gdi32/libgdi32.cross.a ../../dlls/advapi32/libadvapi32.cross.a ../../dlls/ucrtbase/libucrtbase.cross.a 
> make[2]: Leaving directory '/<<PKGBUILDDIR>>/dlls/t2embed'
> x86_64-w64-mingw32-gcc -c -o line.cross.o line.c -I. -I../../include -I../../include/msvcrt -D__WINESRC__ -D_UCRT -DWINE_CROSS_PE -Wall -Wdate-time -g -O2 -fdebug-prefix-map=/<<PKGBUILDDIR>>=. -Wformat -Werror=format-security -Wno-shift-overflow -Wno-unused-function -Wno-deprecated-declarations -fno-strict-aliasing
> In file included from cpanelfolder.c:34:
> In function ‘lstrcpyA’,
>     inlined from ‘IShellExecuteHookA_fnExecute’ at cpanelfolder.c:1080:5:
> ../../include/winbase.h:2854:12: error: ‘strcpy’ offset 10 from the object at ‘pdata’ is out of the bounds of referenced subobject ‘szName’ with type ‘CHAR[1]’ {aka ‘char[1]’} at offset 10 [-Werror=array-bounds]
>  2854 |     return strcpy( dst, src );
>       |            ^~~~~~~~~~~~~~~~~~
> In file included from cpanelfolder.c:44:
> cpanelfolder.c: In function ‘IShellExecuteHookA_fnExecute’:
> pidl.h:119:10: note: subobject ‘szName’ declared here
>   119 |     CHAR szName[1];  /*10*/ /* terminated by 0x00, followed by display name and comment string */
>       |          ^~~~~~
> cc1: all warnings being treated as errors
> make[2]: *** [Makefile:303: cpanelfolder.o] Error 1

The full build log is available from:
   http://qa-logs.debian.net/2020/08/02/wine-development_5.5-5_unstable.log

A list of current common problems and possible solutions is available at
http://wiki.debian.org/qa.debian.org/FTBFS . You're welcome to contribute!

About the archive rebuild: The rebuild was done on EC2 VM instances from
Amazon Web Services, using a clean, minimal and up-to-date chroot. Every
failed build was retried once to eliminate random failures.


Reply to: