mvp: fixed default device detection

The check "open()!=0" is wrong, you have to write "open()>=0", because -1 means error, and 0 is a valid file handle.

mvp: fixed default device detection
dfea6b7c · Max Kellermann · ec926539 · dfea6b7c · dfea6b7c
Commit dfea6b7c authored Feb 26, 2009 by Max Kellermann
Hide whitespace changes
Inline Side-by-side

Showing with 2 additions and 1 deletion

NEWS NEWS +1 -0

mvp_plugin.c src/output/mvp_plugin.c +1 -1

No files found.
--- a/NEWS
+++ b/NEWS
@@ -20,6 +20,7 @@ ver 0.15 - (200?/??/??)
  - mvp: fall back to stereo
  - mvp: fall back to 16 bit audio samples
  - mvp: check for reopen errors
+  - mvp: fixed default device detection
 * commands:
  - "playlistinfo" supports a range now
  - added "sticker database", command "sticker", which allows clients

--- a/src/output/mvp_plugin.c
+++ b/src/output/mvp_plugin.c
@@ -114,7 +114,7 @@ mvp_output_test_default_device(void)

 	fd = open("/dev/adec_pcm", O_WRONLY);

-	if (fd) {
+	if (fd >= 0) {
 		close(fd);
 		return true;
 	}