From 2d4a0e42822172a2ed5d9d1499e17467beb9a4ac Mon Sep 17 00:00:00 2001 From: Johan Hedberg Date: Fri, 18 May 2012 20:25:45 +0300 Subject: [PATCH] sdp: Fix removing servers own record To avoid misbehaving plugins trying to remove handle 0 it's better to directly check for this and return EINVAL. --- src/sdpd-service.c | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/src/sdpd-service.c b/src/sdpd-service.c index c33e1da46..39e05ab18 100644 --- a/src/sdpd-service.c +++ b/src/sdpd-service.c @@ -277,6 +277,10 @@ int remove_record_from_server(uint32_t handle) { sdp_record_t *rec; + /* Refuse to remove the server's own record */ + if (handle == SDP_SERVER_RECORD_HANDLE) + return -EINVAL; + DBG("Removing record with handle 0x%05x", handle); rec = sdp_record_find(handle); -- 2.47.3