mirrors/gstreamer
mirrors/gstreamer
1
1
Fork 0
mirror of https://gitlab.freedesktop.org/gstreamer/gstreamer.git synced 2025-01-11 09:55:36 +00:00
Code Issues 1 Projects Releases Wiki Activity

gdp: Don't write to the same region of memory as a uint64 and uint16 as this breaks ...

Browse source 
Original commit message from CVS:
* libs/gst/dataprotocol/dataprotocol.c:
Don't write to the same region of memory as a uint64 and uint16
as this breaks strict aliasing rules and apparantly breaks on PPC
and s390. Thanks to Sjoerd Simons for analysing. Fixes bug #348114.
This commit is contained in:
Sebastian Dröge 2008-06-30 09:38:45 +00:00 committed by Tim-Philipp Müller
parent 458f65fbba
commit 1774354a3a
1 changed files with 1 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
gst/gdp/dataprotocol.c
View file

@ -82,7 +82,7 @@ GST_DEBUG_CATEGORY_STATIC (data_protocol_debug);
/* helper macros */
/* write first 6 bytes of header, as well as ABI padding */
/* write first 6 bytes of header */
#define GST_DP_INIT_HEADER(h, version, flags, type) \
G_STMT_START { \
gint maj = 0, min = 0; \
@ -95,9 +95,6 @@ G_STMT_START { \
h[2] = (guint8) flags; \
h[3] = 0; /* padding byte */ \
GST_WRITE_UINT16_BE (h + 4, type); \
\
GST_WRITE_UINT64_BE (h + 42, (guint64) 0); /* ABI padding */ \
GST_WRITE_UINT64_BE (h + 50, (guint64) 0); /* ABI padding */ \
} G_STMT_END
#define GST_DP_SET_CRC(h, flags, payload, length); \