From: Djalal Harouni <tixxdz@opendz.org>
Date: Wed, 21 Jan 2015 12:33:37 +0000 (+0100)
Subject: match: for KDBUS_CMD_MATCH_REMOVE fail with EBADSLT for invalid match entries
X-Git-Tag: upstream/0.20150129.081441utc~27
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=915ac894aff954366a96f132a85918f91530edff;p=platform%2Fcore%2Fsystem%2Fkdbus-bus.git

match: for KDBUS_CMD_MATCH_REMOVE fail with EBADSLT for invalid match entries

For KDBUS_CMD_MATCH_REMOVE, if a match entry is not found or invalid
fail with EBADSLT

Signed-off-by: Djalal Harouni <tixxdz@opendz.org>
---

diff --git a/kdbus.txt b/kdbus.txt
index 2be88a9..c149ccc 100644
--- a/kdbus.txt
+++ b/kdbus.txt
@@ -2104,7 +2104,7 @@ For KDBUS_CMD_MATCH_ADD:
 For KDBUS_CMD_MATCH_REMOVE:
 
   -EINVAL	Illegal flags
-  -ENOENT	A match entry with the given cookie could not be found.
+  -EBADSLT	A match entry with the given cookie could not be found.
 
 
 15. Internal object relations
diff --git a/match.c b/match.c
index aae9a85..b1b3731 100644
--- a/match.c
+++ b/match.c
@@ -326,7 +326,7 @@ static int kdbus_match_db_remove_unlocked(struct kdbus_match_db *mdb,
 			found = true;
 		}
 
-	return found ? 0 : -ENOENT;
+	return found ? 0 : -EBADSLT;
 }
 
 /**
diff --git a/test/test-race.c b/test/test-race.c
index b159711..ce8621a 100644
--- a/test/test-race.c
+++ b/test/test-race.c
@@ -284,7 +284,7 @@ TEST_RACE2(kdbus_test_race_byebye_match, 10000,
 		cmd.size = sizeof(cmd);
 		cmd.cookie = 0xdeadbeef;
 		ret = ioctl(env->conn->fd, KDBUS_CMD_MATCH_REMOVE, &cmd);
-		if (ret == 0 || errno == ENOENT)
+		if (ret == 0 || errno == EBADSLT)
 			return 0;
 
 		return -errno;