Skip to content

Native build testing code#22

Open
GoutamD2905 wants to merge 1 commit into
developfrom
feature/coverity_check
Open

Native build testing code#22
GoutamD2905 wants to merge 1 commit into
developfrom
feature/coverity_check

Conversation

@GoutamD2905

Copy link
Copy Markdown
Contributor

No description provided.

Copilot AI review requested due to automatic review settings May 25, 2026 06:06
@GoutamD2905 GoutamD2905 requested review from a team as code owners May 25, 2026 06:06
if(ret < 4)
{
free(pDnsTableEntry);
CcspTraceWarning(("%s skipping invalid entry for mac: %s\n", __FUNCTION__, pDnsTableEntry->MacAddress));

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR adds additional runtime logging while parsing XDNS (dnsmasq) configuration entries, seemingly to support/native-build testing and diagnostics.

Changes:

  • Added an info trace when processing each XDNS entry line.
  • Added a warning trace when an XDNS entry is considered invalid by the parser.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines +1271 to +1276
char *pEntryLog = (char *)malloc(512);
if (pEntryLog)
{
snprintf(pEntryLog, 512, "Processing XDNS entry: %s", buf);
CcspTraceInfo(("%s\n", pEntryLog));
}
Comment on lines 1302 to +1303
free(pDnsTableEntry);
CcspTraceWarning(("%s skipping invalid entry for mac: %s\n", __FUNCTION__, pDnsTableEntry->MacAddress));
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants