Skip to content

[DO NOT MERGE] Dummy change to check coverity#28

Open
Nithishkumar-T wants to merge 1 commit into
developfrom
feature/coverity_dummy
Open

[DO NOT MERGE] Dummy change to check coverity#28
Nithishkumar-T wants to merge 1 commit into
developfrom
feature/coverity_dummy

Conversation

@Nithishkumar-T

Copy link
Copy Markdown
Contributor

No description provided.

Copilot AI review requested due to automatic review settings May 29, 2026 04:15
@Nithishkumar-T Nithishkumar-T requested review from a team as code owners May 29, 2026 04:15

/* HIGH SEVERITY: Buffer overflow (CWE-120) */
char small_buffer[10];
strcpy(small_buffer, "This string is way too long for the buffer"); /* Coverity: BUFFER_SIZE */

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 a deliberately defective Coverity test function to OvsAgent.c to exercise static analysis detection. The PR is explicitly marked [DO NOT MERGE], and the added code is not suitable for production source.

Changes:

  • Adds an uncalled static Coverity dummy function.
  • Seeds intentional defects including null dereference, buffer overflow, resource leak, uninitialized read, unused value, and unreachable code.

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

* This function contains intentional defects for Coverity static analysis testing.
* It is never called and does not affect any functionality.
*/
static void coverity_test_dummy_function_do_not_call(void)
{
/* HIGH SEVERITY: NULL pointer dereference (CWE-476) */
char *null_ptr = NULL;
int high_severity_null_deref = *null_ptr; /* Coverity: FORWARD_NULL */

/* LOW SEVERITY: Dead code - unreachable code after return */
return;
int dead_code_var = 100; /* Coverity: UNREACHABLE */
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