From owner-freebsd-bluetooth@FreeBSD.ORG Sun Nov 4 21:17:36 2007 Return-Path: Delivered-To: freebsd-bluetooth@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id DAEB316A417 for ; Sun, 4 Nov 2007 21:17:36 +0000 (UTC) (envelope-from maksim.yevmenkin@gmail.com) Received: from nf-out-0910.google.com (nf-out-0910.google.com [64.233.182.191]) by mx1.freebsd.org (Postfix) with ESMTP id 21FFC13C4B9 for ; Sun, 4 Nov 2007 21:17:35 +0000 (UTC) (envelope-from maksim.yevmenkin@gmail.com) Received: by nf-out-0910.google.com with SMTP id b2so992145nfb for ; Sun, 04 Nov 2007 13:17:24 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=beta; h=domainkey-signature:received:received:message-id:date:from:to:subject:cc:in-reply-to:mime-version:content-type:content-transfer-encoding:content-disposition:references; bh=xl4PzjzA7y42PqD53Lr2L5EYVD89E/VqIou2pO+pIL8=; b=G9tcfkqtcNN0A4KRHY/KEhtJD/knV6/lYvm8yCTFX+aaUacd26FayxHHsey2Zxm6vIQ4MrVKe5b2y9tzzokPPy/Nb2uxwFxssMJ1tMpSlszzY31SxYQl/SO1qAGnu4tp/wJvjqYuyvu4yTtlw75+0Hf1EYrZknXKOrf10IiRUpM= DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=beta; h=received:message-id:date:from:to:subject:cc:in-reply-to:mime-version:content-type:content-transfer-encoding:content-disposition:references; b=PuIKAt9mOaYt/TNwugUex+fhiYHHuF14TERFeYx6iy/VFnmE9kcjx/92WyOn16heiuu7IaKr2BioKVSK6vlcCcfdByNzlQ9Q+QWg+XPJislfLiz8dml+SfGb1QI308Tb0FUIelqlltpoVvixNuDagitGOJ2yLC3/4NJc0EOWRBU= Received: by 10.86.74.15 with SMTP id w15mr2826766fga.1194211044654; Sun, 04 Nov 2007 13:17:24 -0800 (PST) Received: by 10.86.83.12 with HTTP; Sun, 4 Nov 2007 13:17:24 -0800 (PST) Message-ID: Date: Sun, 4 Nov 2007 14:17:24 -0700 From: "Maksim Yevmenkin" To: "Iain Hibbert" In-Reply-To: <1194174661.309391.11026.nullmailer@galant.ukfsn.org> MIME-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Content-Disposition: inline References: <1194174661.309391.11026.nullmailer@galant.ukfsn.org> Cc: freebsd-bluetooth@freebsd.org Subject: Re: sdp - SearchService of UUID X-BeenThere: freebsd-bluetooth@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: Using Bluetooth in FreeBSD environments List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sun, 04 Nov 2007 21:17:36 -0000 Iain, i think, but not 100% sure yet, you might be correct. the spec says: "A service search pattern is said to match a service record if each and every UUID in the service search pattern is contained within any of the service record's attribute values." later "The only time a service search pattern does not match a service record is if the service search pattern contains at least one UUID that is not contained within the service record's attribute values." so, i think, the sdp query from your "htc touch" phone says "give me all the service records that contain uuid for obex protocol". in other words, such query should return both obex object push and obex file transfer services if both are registered. i also must say, i have never seen this before :) so, provider_match_uuid() should probably accept the list of all uuids from the search pattern (we can limit it to 12 as spec says we should not expect more) and match all uuids from search pattern against uuids in provider's service record. i think your general idea, i.e. keeping a list of uuids instead of one uuid is correct, however, implementation needs to be slightly different (hmm... i wonder if linux-bluez sdp has the same problem). and, just to be sure, i made similar (i.e. search pattern contained only obex uuid) request to my se k700i and got a bunch of service records back. so, we have to fix our sdpd. thanks, max On 11/4/07, Iain Hibbert wrote: > Hi Max, > > I just got a new phone (HTC Touch) with Windows Mobile 6.0 on it (bah!) > and am having trouble sending files with Bluetooth to my computer, though > it manages to send fine to a Nokia 6103 which also sends fine to my > computer, and I can use obexapp to access the FTRN service on the Touch > and send and receive files that way. > > The problem seems to be that it sends a SearchServiceAttribute Request > with SDP_UUID_PROTOCOL_OBEX instead of a service class, like so: > > > ACL data: handle 45 flags 0x02 dlen 25 > L2CAP(d): cid 0x0044 len 21 [psm 1] > SDP SSA Req: tid 0x0 len 0x10 > pat uuid-16 0x0008 (OBEX) > max 4088 > aid(s) 0x0001 (SrvClassIDList) 0x0004 (ProtocolDescList) > cont 00 > > and we reply with "no attributes found": > > < ACL data: handle 45 flags 0x02 dlen 15 > L2CAP(d): cid 0x004d len 11 [psm 1] > SDP SSA Rsp: tid 0x0 len 0x6 > count 3 > cont 00 > > so it gives up. In contrast, the Nokia sends a ServiceSearchRequest for > ObjectPUSH then ServiceAttributeRequest for the handle and all works > fine.. > > So, I'm looking at the SDP spec and I'm not finding it really clear but it > seems (section 2.7.2) that a ServiceSearch should cover any UUID within > the record (?) not just the service class but I think that sdpd does not > do that, as it only keeps a single UUID per profile.. > > Since we don't have an actual record in memory because we build the > appropriate parts when requested (?), I've come up with the following > [incomplete*] scheme to extend the profile uuid into a uuid list and made > a provider_match_uuid() function for the ServiceSearch and > ServiceSearchAttribute functions to use. > > what do you think, is this the right way to go? > > regards, > iain > > * > - only compile tested so far > - needs further study of specs to see which UUIDs should be in a profile > > --- /usr/src/usr.sbin/sdpd/bgd.c 2006-06-19 16:44:56.000000000 +0100 > +++ bgd.c 2007-11-04 10:30:24.000000000 +0000 > @@ -98,8 +98,13 @@ > { 0, NULL } /* end entry */ > }; > > -profile_t bgd_profile_descriptor = { > +static uint16_t bgd_profile_uuids[] = { > SDP_SERVICE_CLASS_BROWSE_GROUP_DESCRIPTOR, > +}; > + > +profile_t bgd_profile_descriptor = { > + bgd_profile_uuids, > + nelems(bgd_profile_uuids), > 0, > (profile_data_valid_p) NULL, > (attr_t const * const) &bgd_profile_attrs > --- /usr/src/usr.sbin/sdpd/dun.c 2006-06-19 16:44:56.000000000 +0100 > +++ dun.c 2007-11-04 10:54:04.000000000 +0000 > @@ -132,8 +132,16 @@ > { 0, NULL } /* end entry */ > }; > > -profile_t dun_profile_descriptor = { > +static uint16_t dun_profile_uuids[] = { > SDP_SERVICE_CLASS_DIALUP_NETWORKING, > + SDP_SERVICE_CLASS_GENERIC_NETWORKING, > + SDP_UUID_PROTOCOL_L2CAP, > + SDP_UUID_PROTOCOL_RFCOMM, > + 0x0100, > +}; > + > +profile_t dun_profile_descriptor = { > + dun_profile_uuids, nelems(dun_profile_uuids), > sizeof(sdp_dun_profile_t), > common_profile_server_channel_valid, > (attr_t const * const) &dun_profile_attrs > --- /usr/src/usr.sbin/sdpd/ftrn.c 2006-06-19 16:44:56.000000000 +0100 > +++ ftrn.c 2007-11-04 10:56:08.000000000 +0000 > @@ -113,8 +113,15 @@ > { 0, NULL } /* end entry */ > }; > > -profile_t ftrn_profile_descriptor = { > +static uint16_t ftrn_profile_uuids[] = { > SDP_SERVICE_CLASS_OBEX_FILE_TRANSFER, > + SDP_UUID_PROTOCOL_L2CAP, > + SDP_UUID_PROTOCOL_RFCOMM, > + SDP_UUID_PROTOCOL_OBEX, > +}; > + > +profile_t ftrn_profile_descriptor = { > + ftrn_profile_uuids, nelems(ftrn_profile_uuids), > sizeof(sdp_ftrn_profile_t), > common_profile_server_channel_valid, > (attr_t const * const) &ftrn_profile_attrs > --- /usr/src/usr.sbin/sdpd/hf.c 2006-08-17 21:13:31.000000000 +0100 > +++ hf.c 2007-11-04 10:52:14.000000000 +0000 > @@ -148,8 +148,17 @@ > { 0, NULL } /* end entry */ > }; > > -profile_t hf_profile_descriptor = { > +static uint16_t hf_profile_uuids[] = { > SDP_SERVICE_CLASS_HANDSFREE, > + SDP_SERVICE_CLASS_HANDSFREE_AUDIO_GATEWAY, > + SDP_SERVICE_CLASS_GENERIC_AUDIO, > + SDP_UUID_PROTOCOL_L2CAP, > + SDP_UUID_PROTOCOL_RFCOMM, > + 0x0105, > +}; > + > +profile_t hf_profile_descriptor = { > + hf_profile_uuids, nelems(hf_profile_uuids), > sizeof(sdp_hf_profile_t), > hf_profile_data_valid, > (attr_t const * const) &hf_profile_attrs > --- /usr/src/usr.sbin/sdpd/hset.c 2006-06-19 16:44:56.000000000 +0100 > +++ hset.c 2007-11-04 10:57:24.000000000 +0000 > @@ -128,8 +128,16 @@ > { 0, NULL } /* end entry */ > }; > > -profile_t hset_profile_descriptor = { > +static uint16_t hset_profile_uuids[] = { > SDP_SERVICE_CLASS_HEADSET_AUDIO_GATEWAY, > + SDP_SERVICE_CLASS_GENERIC_AUDIO, > + SDP_SERVICE_CLASS_HEADSET, > + SDP_UUID_PROTOCOL_L2CAP, > + SDP_UUID_PROTOCOL_RFCOMM, > +}; > + > +profile_t hset_profile_descriptor = { > + hset_profile_uuids, nelems(hset_profile_uuids), > sizeof(sdp_hset_profile_t), > hset_profile_data_valid, > (attr_t const * const) &hset_profile_attrs > --- /usr/src/usr.sbin/sdpd/irmc.c 2006-06-19 16:44:56.000000000 +0100 > +++ irmc.c 2007-11-04 10:31:15.000000000 +0000 > @@ -129,8 +129,12 @@ > { 0, NULL } /* end entry */ > }; > > -profile_t irmc_profile_descriptor = { > +static uint16_t irmc_profile_uuids[] = { > SDP_SERVICE_CLASS_IR_MC_SYNC, > +}; > + > +profile_t irmc_profile_descriptor = { > + irmc_profile_uuids, nelems(irmc_profile_uuids), > sizeof(sdp_irmc_profile_t), > obex_profile_data_valid, > (attr_t const * const) &irmc_profile_attrs > --- /usr/src/usr.sbin/sdpd/irmc_command.c 2006-06-19 16:44:56.000000000 +0100 > +++ irmc_command.c 2007-11-04 10:31:27.000000000 +0000 > @@ -113,8 +113,12 @@ > { 0, NULL } /* end entry */ > }; > > -profile_t irmc_command_profile_descriptor = { > +static uint16_t irmc_command_profile_uuids[] = { > SDP_SERVICE_CLASS_IR_MC_SYNC_COMMAND, > +}; > + > +profile_t irmc_command_profile_descriptor = { > + irmc_command_profile_uuids, nelems(irmc_command_profile_uuids), > sizeof(sdp_irmc_command_profile_t), > common_profile_server_channel_valid, > (attr_t const * const) &irmc_command_profile_attrs > --- /usr/src/usr.sbin/sdpd/lan.c 2006-06-19 16:44:56.000000000 +0100 > +++ lan.c 2007-11-04 10:31:36.000000000 +0000 > @@ -173,8 +173,12 @@ > { 0, NULL } /* end entry */ > }; > > -profile_t lan_profile_descriptor = { > +static uint16_t lan_profile_uuids[] = { > SDP_SERVICE_CLASS_LAN_ACCESS_USING_PPP, > +}; > + > +profile_t lan_profile_descriptor = { > + lan_profile_uuids, nelems(lan_profile_uuids), > sizeof(sdp_lan_profile_t), > lan_profile_data_valid, > (attr_t const * const) &lan_profile_attrs > --- /usr/src/usr.sbin/sdpd/opush.c 2006-06-19 16:44:56.000000000 +0100 > +++ opush.c 2007-11-04 10:58:30.000000000 +0000 > @@ -129,8 +129,15 @@ > { 0, NULL } /* end entry */ > }; > > -profile_t opush_profile_descriptor = { > +static uint16_t opush_profile_uuids[] = { > SDP_SERVICE_CLASS_OBEX_OBJECT_PUSH, > + SDP_UUID_PROTOCOL_L2CAP, > + SDP_UUID_PROTOCOL_RFCOMM, > + SDP_UUID_PROTOCOL_OBEX, > +}; > + > +profile_t opush_profile_descriptor = { > + opush_profile_uuids, nelems(opush_profile_uuids), > sizeof(sdp_opush_profile_t), > obex_profile_data_valid, > (attr_t const * const) &opush_profile_attrs > --- /usr/src/usr.sbin/sdpd/profile.c 2006-08-19 18:11:04.000000000 +0100 > +++ profile.c 2007-11-04 10:20:55.000000000 +0000 > @@ -41,6 +41,7 @@ > #include > #include "profile.h" > #include "provider.h" > +#include "uuid-private.h" > > /* > * Lookup profile descriptor > @@ -74,7 +75,7 @@ > int32_t i; > > for (i = 0; i < sizeof(profiles)/sizeof(profiles[0]); i++) > - if (profiles[i]->uuid == uuid) > + if (profiles[i]->uuid[0] == uuid) > return (profiles[i]); > > return (NULL); > --- /usr/src/usr.sbin/sdpd/profile.h 2006-06-19 16:44:56.000000000 +0100 > +++ profile.h 2007-11-04 10:37:28.000000000 +0000 > @@ -63,7 +63,8 @@ > > struct profile > { > - uint16_t uuid; /* profile uuid */ > + uint16_t * uuid; /* profile uuid list */ > + int ucount; /* profile uuid count */ > uint16_t dsize; /* profile data size */ > profile_data_valid_p valid; /* profile data validator */ > attr_t const * const attrs; /* supported attributes */ > @@ -88,4 +89,6 @@ > profile_data_valid_t common_profile_server_channel_valid; > profile_data_valid_t obex_profile_data_valid; > > +#define nelems(list) (sizeof(list) / sizeof(list[0])) > + > #endif /* ndef _PROFILE_H_ */ > --- /usr/src/usr.sbin/sdpd/provider.c 2006-06-19 16:44:56.000000000 +0100 > +++ provider.c 2007-11-04 10:40:54.000000000 +0000 > @@ -36,10 +36,12 @@ > > #include > #include > +#include > #include > #include > #include "profile.h" > #include "provider.h" > +#include "uuid-private.h" > > static TAILQ_HEAD(, provider) providers = TAILQ_HEAD_INITIALIZER(providers); > static uint32_t change_state = 0; > @@ -198,3 +200,28 @@ > { > return (change_state); > } > + > +/* > + * Match provider by uuid > + */ > + > +int > +provider_match_uuid(provider_p provider, uint128_t *uuid) > +{ > + uint128_t puuid; > + int i; > + > + if (memcmp(uuid, &uuid_public_browse_group, sizeof(*uuid)) == 0) > + return 1; > + > + for (i = 0 ; i < provider->profile->ucount ; i++) { > + memcpy(&puuid, &uuid_base, sizeof(puuid)); > + puuid.b[2] = provider->profile->uuid[i] >> 8; > + puuid.b[3] = provider->profile->uuid[i]; > + > + if (memcmp(uuid, &puuid, sizeof(*uuid)) == 0) > + return 1; > + } > + > + return 0; > +} > --- /usr/src/usr.sbin/sdpd/provider.h 2006-06-19 16:44:56.000000000 +0100 > +++ provider.h 2007-11-04 09:40:59.000000000 +0000 > @@ -100,6 +100,8 @@ > provider_p provider_get_first (void); > provider_p provider_get_next (provider_p provider); > uint32_t provider_get_change_state (void); > +int provider_match_uuid (provider_p provider, > + uint128_t *uuid); > > int32_t server_prepare_attr_list(provider_p const, uint8_t const *, uint8_t const *, > uint8_t *, uint8_t const *); > --- /usr/src/usr.sbin/sdpd/sd.c 2006-06-19 16:44:56.000000000 +0100 > +++ sd.c 2007-11-04 10:55:05.000000000 +0000 > @@ -208,8 +208,13 @@ > { 0, NULL } /* end entry */ > }; > > -profile_t sd_profile_descriptor = { > +static uint16_t sd_profile_uuids[] = { > SDP_SERVICE_CLASS_SERVICE_DISCOVERY_SERVER, > + SDP_UUID_PROTOCOL_L2CAP, > +}; > + > +profile_t sd_profile_descriptor = { > + sd_profile_uuids, nelems(sd_profile_uuids), > 0, > (profile_data_valid_p) NULL, > (attr_t const * const) &sd_profile_attrs > --- /usr/src/usr.sbin/sdpd/sp.c 2006-06-19 16:44:56.000000000 +0100 > +++ sp.c 2007-11-04 10:59:04.000000000 +0000 > @@ -113,8 +113,14 @@ > { 0, NULL } /* end entry */ > }; > > -profile_t sp_profile_descriptor = { > +static uint16_t sp_profile_uuids[] = { > SDP_SERVICE_CLASS_SERIAL_PORT, > + SDP_UUID_PROTOCOL_L2CAP, > + SDP_UUID_PROTOCOL_RFCOMM, > +}; > + > +profile_t sp_profile_descriptor = { > + sp_profile_uuids, nelems(sp_profile_uuids), > sizeof(sdp_sp_profile_t), > common_profile_server_channel_valid, > (attr_t const * const) &sp_profile_attrs > --- /usr/src/usr.sbin/sdpd/ssar.c 2006-06-19 16:44:56.000000000 +0100 > +++ ssar.c 2007-11-04 09:41:01.000000000 +0000 > @@ -60,7 +60,7 @@ > > provider_t *provider = NULL; > int32_t type, rsp_limit, ssplen, aidlen, cslen, cs; > - uint128_t uuid, puuid; > + uint128_t uuid; > > /* > * Minimal Service Search Attribute Request request > @@ -217,12 +217,7 @@ > if (!provider_match_bdaddr(provider, &srv->req_sa.bt_bdaddr)) > continue; > > - memcpy(&puuid, &uuid_base, sizeof(puuid)); > - puuid.b[2] = provider->profile->uuid >> 8; > - puuid.b[3] = provider->profile->uuid; > - > - if (memcmp(&uuid, &puuid, sizeof(uuid)) != 0 && > - memcmp(&uuid, &uuid_public_browse_group, sizeof(uuid)) != 0) > + if (!provider_match_uuid(provider, &uuid)) > continue; > > cs = server_prepare_attr_list(provider, > --- /usr/src/usr.sbin/sdpd/ssr.c 2006-06-19 16:44:56.000000000 +0100 > +++ ssr.c 2007-11-04 09:41:04.000000000 +0000 > @@ -63,7 +63,7 @@ > uint8_t *ptr = NULL; > provider_t *provider = NULL; > int32_t type, ssplen, rsp_limit, rcount, cslen, cs; > - uint128_t uuid, puuid; > + uint128_t uuid; > > /* > * Minimal SDP Service Search Request > @@ -190,15 +190,11 @@ > if (!provider_match_bdaddr(provider, &srv->req_sa.bt_bdaddr)) > continue; > > - memcpy(&puuid, &uuid_base, sizeof(puuid)); > - puuid.b[2] = provider->profile->uuid >> 8; > - puuid.b[3] = provider->profile->uuid; > - > - if (memcmp(&uuid, &puuid, sizeof(uuid)) == 0 || > - memcmp(&uuid, &uuid_public_browse_group, sizeof(uuid)) == 0) { > - SDP_PUT32(provider->handle, ptr); > - rcount ++; > - } > + if (!provider_match_uuid(provider, &uuid)) > + continue; > + > + SDP_PUT32(provider->handle, ptr); > + rcount ++; > } > } > > _______________________________________________ > freebsd-bluetooth@freebsd.org mailing list > http://lists.freebsd.org/mailman/listinfo/freebsd-bluetooth > To unsubscribe, send any mail to "freebsd-bluetooth-unsubscribe@freebsd.org" >