ports/139384: audio/murmur fails to work in UDP mode on amd64

[Richard Tector]

>Number:         139384
>Category:       ports
>Synopsis:       audio/murmur fails to work in UDP mode on amd64
>Confidential:   no
>Severity:       serious
>Priority:       medium
>Responsible:    freebsd-ports-bugs
>State:          open
>Quarter:        
>Keywords:       
>Date-Required:
>Class:          sw-bug
>Submitter-Id:   current-users
>Arrival-Date:   Tue Oct 06 22:50:00 UTC 2009
>Closed-Date:
>Last-Modified:
>Originator:     Richard Tector
>Release:        FreeBSD 8.0-RC1
>Organization:
>Environment:
FreeBSD coyote.tector.org.uk 8.0-RC1 FreeBSD 8.0-RC1 #6: Tue Sep 29 13:00:54 BST 2009     richard at coyote.tector.org.uk:/usr/obj/usr/src/sys/COYOTE  amd64
>Description:
A bug in the murmur crypto code means that on amd64 (and possibly other 64bit architectures) mumble clients do not work correctly using the default and preferred UDP transport for voice traffic, and must instead use TCP mode.

The problem was discussed at:
https://sourceforge.net/projects/mumble/forums/forum/492607/topic/3378871


Further, files/patch-comipler.pri typo should be fixed to patch-compiler.pri
>How-To-Repeat:
Install murmur on 64bit system, try to connect using UDP with mumble. Voice is transmitted but ignored by murmur.
>Fix:
While this has been corrected upstream for the coming 1.2.0 release, the bug still exists in 1.1.8 installed by the port. The attached patch fixes the problem.

Patch attached with submission follows:

diff -uNr audio/murmur.orig/files/patch-src__CryptState.cpp audio/murmur/files/patch-src__CryptState.cpp
--- audio/murmur.orig/files/patch-src__CryptState.cpp	1970-01-01 01:00:00.000000000 +0100
+++ audio/murmur/files/patch-src__CryptState.cpp	2009-10-06 23:33:56.000000000 +0100
@@ -0,0 +1,11 @@
+--- src/CryptState.cpp.orig	2009-03-22 15:05:00.000000000 +0000
++++ src/CryptState.cpp	2009-10-06 21:19:31.000000000 +0100
+@@ -187,7 +187,7 @@
+ #define SHIFTBITS 63
+ typedef quint64 subblock;
+ 
+-#if __BYTE_ORDER == __BIG_ENDIAN
++#if Q_BYTE_ORDER == Q_BIG_ENDIAN
+ #define SWAPPED(x) (x)
+ #else
+ #ifdef __x86_64__


>Release-Note:
>Audit-Trail:
>Unformatted: