gcoap_dns: various fixes [backport 2022.07] #18373
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Backport of #18367
Contribution description
While doing some more experiments with the current version of
gcoap_dns
, I noticed some bugs, specifically, when using block-wise transfer. This PR fixes those and also replaces some left-overprintf
s withDEBUG
.Testing procedure
Similar to the testing procedures in #16705:
Wiershark also shows blockwise transfer for both request and response:
Without this fix
query example.org inet6
will just fail withUnable to resolve query: Invalid argument
(stemming from the wrongly reported-EINVAL
).Issues/PRs references
Follow-up on #16705