tree ffd0d636f771df76d632f19996bf02e0242c1371
parent d2de5d5bcb7a31e282444d8a1399d4c1da2f438e
author Damian Królik <66667989+Damian-Nordic@users.noreply.github.com> 1704325647 +0100
committer GitHub <noreply@github.com> 1704325647 +0000
gpgsig -----BEGIN PGP SIGNATURE-----
 
 wsBcBAABCAAQBQJllfIPCRBK7hj4Ov3rIwAAraIIAIAYSIMRKhQfZFy2ujrxo1JD
 hEs4jshA2V7BLQF3FSqjq+94WwXSenZ7cn97cFuarg6xNZ5BvDkZQLZ6uAop7oNL
 jUr6hbtRIoEm29v8SuewG8wKAsH/soqF/Qsjs7Ad0Lx8FOwK1+fW4M+F+6SHECyx
 Rh2aWyDAcYGFEPlQ0hVy8Gdw5BuJYlfYEqRolLja/WagP1rAYhGpsMVOgG5q1908
 m6NhfGouPWdeHX+ysD2p1fWKC9k/PetN0lNnbwRr57N3RVkma/1asf9yzgMtvVBu
 2zzs0cZwiFvJnt0b6kaAjuMP0cfhB+C/TA/DvFp2Tl0ly9KeP/Os2gwCGSahgt0=
 =hHgw
 -----END PGP SIGNATURE-----
 

[OpenThread] Harden DNS record parsing (#31227)

OpenThread applications would crash upon receiving an empty
DNS TXT record. The reason was that the code for copying OT
DNS service info object into Matter DnssdService object
would not initialize the TXT entry count in the latter
object in such a case.

In the reported case, the Matter stack was presented an
empty TXT record because OpenThread's DNS client received
a TXT record with TTL 0 and it discarded its contents.
Nevertheless, the issue could be reproduced by publishing
Matter service without TXT entries and kicking off DNS query.

1. Initialize the TXT entry and subtype count properly in all
   scenarios.
2. Do not even process the service info object if an error was
   returned by OpenThread before.
3. Extract some boilerplate to a separate function to improve
   readability.

Signed-off-by: Damian Krolik <damian.krolik@nordicsemi.no>