Skip to content

Commit 1743980

Browse files
cecillewoody-appletcarmelveilleux
authored andcommitted
Fix short code on dnssd advertising and chip-tool (#12319)
* Fix short code on dnssd advertising and chip-tool The short discriminator is the UPPER four bits of the discriminator. * Update src/controller/SetUpCodePairer.cpp Co-authored-by: Tennessee Carmel-Veilleux <tennessee.carmelveilleux@gmail.com> * Update src/app/server/Dnssd.cpp Co-authored-by: Tennessee Carmel-Veilleux <tennessee.carmelveilleux@gmail.com> Co-authored-by: Justin Wood <woody@apple.com> Co-authored-by: Tennessee Carmel-Veilleux <tennessee.carmelveilleux@gmail.com>
1 parent e3bf329 commit 1743980

2 files changed

Lines changed: 3 additions & 2 deletions

File tree

src/app/server/Dnssd.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -320,7 +320,7 @@ CHIP_ERROR DnssdServer::Advertise(bool commissionableNode, chip::Dnssd::Commissi
320320
ChipLogError(Discovery, "Setup discriminator not known. Using a default.");
321321
value = 840;
322322
}
323-
advertiseParameters.SetShortDiscriminator(static_cast<uint8_t>(value & 0xFF)).SetLongDiscriminator(value);
323+
advertiseParameters.SetShortDiscriminator(static_cast<uint8_t>((value >> 8) & 0x0F)).SetLongDiscriminator(value);
324324

325325
if (DeviceLayer::ConfigurationMgr().IsCommissionableDeviceTypeEnabled() &&
326326
DeviceLayer::ConfigurationMgr().GetDeviceTypeId(value) == CHIP_NO_ERROR)

src/controller/SetUpCodePairer.cpp

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -64,7 +64,8 @@ CHIP_ERROR SetUpCodePairer::Connect(RendezvousInformationFlag rendezvousInformat
6464

6565
if (searchOverAll || rendezvousInformation == RendezvousInformationFlag::kOnNetwork)
6666
{
67-
if (CHIP_NO_ERROR == (err = StartDiscoverOverIP(discriminator, isShort)))
67+
if (CHIP_NO_ERROR ==
68+
(err = StartDiscoverOverIP(isShort ? static_cast<uint16_t>((discriminator >> 8) & 0x0F) : discriminator, isShort)))
6869
{
6970
isRunning = true;
7071
}

0 commit comments

Comments
 (0)