Commit: 6e62d4a0fa33968b6c7a8e0d543bca2cdac323ed
Parent: 7b1bda3ac6728e36f476b94c2e04128e4a6a6628
Author: Mikel Astiz <mikel.astiz@bmw-carit.de>
Committer: Luiz Augusto von Dentz <luiz.von.dentz@intel.com>
Date: 2012-09-14 15:09:17
Tree: 5b05d459db60643a558cfc1419050d7e849ec550

media: Fix accesstype comparison Replace the string representation of the accesstype with a conventional binary representation. This makes the code simpler and more efficient. This also fixes a minor bug in the Release() D-Bus method, where the string comparison was used to see whether the owner should be removed. A client acquiring with "rw" and releasing with "wr" would lead to the inconsistent state of having a released transport with an owner with no accesstype. Partial releases can also get affected by this bug since the released character (partial accesstype) got replaced by a whitespace. Additionally, this approach is more robust in case new flags are added in the future.

Diffstat

M audio/transport.c | 119 ++++++++++++++++++++++++++++++++++++++++- - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - -

1 files changed, 68 insertions(+), 51 deletions(-)

View Full Diff | Patch