-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/100710/
-----------------------------------------------------------
Review request for Amarok.
Summary
-------
The Firefly version installed on my Readynas Duo (Radiator 4.1.7) sends the
aeMK (mediakind) field as an 32 bit int instead of a 8 bit char.
Amarok currently only consumes one byte instead of four, despite the fact that
the field length is correctly reported as four bytes by the server. The
attached patch reworks the DAAP protocol parsing to be more robust.
This addresses bug 266676.
https://bugs.kde.org/show_bug.cgi?id=266676
Diffs
-----
src/core-impl/collections/daap/daapreader/Reader.h 8f882b8
src/core-impl/collections/daap/daapreader/Reader.cpp 744c6a2
Diff: http://git.reviewboard.kde.org/r/100710/diff
Testing
-------
Yes. With the attached patch Amarok plays music via DAAP from my NAS just fine.
Thanks,
Lukas
_______________________________________________
Amarok-devel mailing list
Amarok-devel@kde.org
https://mail.kde.org/mailman/listinfo/amarok-devel
