Patch to improve DNS answers parser

Patch to improve DNS answers parser

[Leandro G. B. Becker]

[-- Attachment #1.1: Type: text/plain, Size: 1724 bytes --]

Hello folks,

I made an improvement to DNS client (DnsImpl.c, line 1392) that instead of abort processing answer records when a record that is not of type PTR is received, skip the record and goes to the next one if there is one. I had a problem while resolving some names when DNS receives more than one answer and the first one is not of type 0xC0 (PTR) and now it is working fine.

So to submit the patch for review is through this e-mail group, following the instructions at https://github.com/tianocore/edk2, section "Code Contributions"?

Thank you!

LEANDRO GUSTAVO BISS BECKER
Development Engineer
R&D - BIOS Development - Manaus
lbecker@positivo.com.br<mailto:lbecker@positivo.com.br>
Positivo Tecnologia
Tel.: +55 92  3183-7988
[https://intracorporativa.positivo.com.br/assinatura-positivo/img/logos/comp/positivo_tecnologia.jpg]<http://www.positivotecnologia.com.br/>

This message may contain confidential and/or legally privileged information. If you are not the intended recipient or the person authorized to receive this message, you must not use, copy or disclose the information contained herein or take any action based on this content, and you must notify the sender and delete the message permanently from your system.
Positivo Tecnologia seeks to ensure the highest level of corporate integrity and ethics in its activities, making available to all the "Canal Aberto", through which anyone can report possible violations of internal policies, laws and regulations. The "Canal Aberto" can be accessed anonymously, anytime, through the website www.positivotecnologia.com.br/canalaberto <http://www.positivotecnologia.com.br/canalaberto> or by calling 0800 727 7016<tel:08007277016>.



[-- Attachment #1.2: Type: text/html, Size: 8078 bytes --]

[-- Attachment #2: image003.jpg --]
[-- Type: image/jpeg, Size: 1709 bytes --]

Re: [edk2-devel] Patch to improve DNS answers parser

[Laszlo Ersek]

On 08/26/20 18:24, Leandro G. B. Becker wrote:
> Hello folks,
> 
> I made an improvement to DNS client (DnsImpl.c, line 1392) that instead of abort processing answer records when a record that is not of type PTR is received, skip the record and goes to the next one if there is one. I had a problem while resolving some names when DNS receives more than one answer and the first one is not of type 0xC0 (PTR) and now it is working fine.
> 
> So to submit the patch for review is through this e-mail group, following the instructions at https://github.com/tianocore/edk2, section "Code Contributions"?

Yes please!

See also the official description at:

  https://github.com/tianocore/tianocore.github.io/wiki/EDK-II-Development-Process

and if you care for a very long & unofficial one:

  https://github.com/tianocore/tianocore.github.io/wiki/Laszlo's-unkempt-git-guide-for-edk2-contributors-and-maintainers

When you post the patch for NetworkPkg, please CC the NetworkPkg reviewers / maintainers (see Maintainers.txt).

Thanks!
Laszlo