Skip to content

Commit ae381b7

Browse files
mdns: Fix potential null deref reported by fuzzer test
* Original commit: espressif/esp-idf@cb5653f
1 parent d0f4e68 commit ae381b7

File tree

1 file changed

+2
-1
lines changed

1 file changed

+2
-1
lines changed

components/mdns/mdns.c

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2936,7 +2936,8 @@ static bool _mdns_question_matches(mdns_parsed_question_t * question, uint16_t t
29362936
if (type == MDNS_TYPE_A || type == MDNS_TYPE_AAAA) {
29372937
return true;
29382938
} else if (type == MDNS_TYPE_PTR || type == MDNS_TYPE_SDPTR) {
2939-
if (!strcasecmp(service->service->service, question->service)
2939+
if (question->service && question->proto && question->domain
2940+
&& !strcasecmp(service->service->service, question->service)
29402941
&& !strcasecmp(service->service->proto, question->proto)
29412942
&& !strcasecmp(MDNS_DEFAULT_DOMAIN, question->domain)) {
29422943
return true;

0 commit comments

Comments
 (0)