[MPlayer-users] msadpcm crash with Solaris/SPARC

[Götz Waschk]

Am Mittwoch, 23. Oktober 2002, 14:48:15 Uhr MET, schrieb Götz Waschk:
> > > The right one was this:
> > > #define LE_16(x) (le2me_16((x)[1]+(256*(x)[0])))
> > does it work with it?
> Yes, it's working fine.

You've mixed this up in CVS:
Current CVS contains this line:
#define LE_16(x) (le2me_16((x)[0]+(256*((x)[1]))))

but the right working macro is this one:
#define LE_16(x) (le2me_16((x)[1]+(256*((x)[0]))))

Please apply the attached patch.

-- 
   Götz Waschk <> master of computer science  <> University of Rostock    
 http://wwwtec.informatik.uni-rostock.de/~waschk/waschk.asc for PGP key
                         --> Logout Fascism! <--
-------------- next part --------------
Index: libmpcodecs/ad_msadpcm.c
===================================================================
RCS file: /cvsroot/mplayer/main/libmpcodecs/ad_msadpcm.c,v
retrieving revision 1.9
diff -u -r1.9 ad_msadpcm.c
--- libmpcodecs/ad_msadpcm.c	23 Oct 2002 23:48:42 -0000	1.9
+++ libmpcodecs/ad_msadpcm.c	24 Oct 2002 08:07:51 -0000
@@ -44,7 +44,7 @@
 
 #define MS_ADPCM_PREAMBLE_SIZE 6
 
-#define LE_16(x) (le2me_16((x)[0]+(256*((x)[1]))))
+#define LE_16(x) (le2me_16((x)[1]+(256*((x)[0]))))
 //#define LE_16(x) (le2me_16((x)[1]+(256*((x)[0]))))
 //#define LE_16(x) (le2me_16(*(unsigned short *)(x)))
 //#define LE_32(x) (le2me_32(*(unsigned int *)(x)))