Skip to content

Add boundary check before memset, memcpy, strncpy#278

Draft
maipbui wants to merge 1 commit intoAzure:masterfrom
maipbui:add_boundary_check
Draft

Add boundary check before memset, memcpy, strncpy#278
maipbui wants to merge 1 commit intoAzure:masterfrom
maipbui:add_boundary_check

Conversation

@maipbui
Copy link
Member

@maipbui maipbui commented Mar 4, 2024

ADO 27008041

Why I did it

Add boundary check before memset, memcpy, strncpy calls to prevent buffer overflow

How I did it

How to verify it

Which release branch to backport (provide reason below if selected)

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106
  • 202111
  • 202205
  • 202305

Description for the changelog

A picture of a cute animal (not mandatory but encouraged)

Signed-off-by: Mai Bui <maibui@microsoft.com>
@maipbui maipbui requested a review from qiluo-msft March 4, 2024 17:55
@maipbui maipbui requested a review from lguohan as a code owner March 4, 2024 17:55
@maipbui maipbui marked this pull request as draft March 5, 2024 18:02
liushilongbuaa pushed a commit that referenced this pull request Mar 25, 2026
…utomatically (#24404)

#### Why I did it
src/sonic-host-services
```
* fda4e65 - (HEAD -> master, origin/master, origin/HEAD) Include info level logs in reboot cause services (#278) (7 days ago) [roy-nexthop]
* 3492186 - Module graceful shutdown support commit 255 is triggering loganalyzer failures in non-smartswitch platforms (#322) (8 days ago) [rameshraghupathy]
* 1d7ec58 - Remove unwrap() in sonic-host-services (#321) (9 days ago) [Qi Luo]
* bbee5e6 - Call sonic-kdump-config --remote after setting ssh_path and ssh_string (#319) (12 days ago) [HP]
* ce60800 - Ignore failures in enabling a systemd service (#320) (12 days ago) [Saikrishna Arcot]
* ee9ef3f - Module graceful shutdown support (#255) (13 days ago) [rameshraghupathy]
* f1de8e3 - Add ExclusionList for telemetry/frr_bmp since they are not service in pub repo (#317) (3 weeks ago) [Feng-msft]
* 6507fa3 - [hostcfg] Fix timezone mismatch after image upgrade (#312) (4 weeks ago) [Ben Levi]
```
#### How I did it
#### How to verify it
#### Description for the changelog
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.

1 participant