Skip to content

Fix NULL pointer access in systemd-sonic-generator#154

Closed
stephenxs wants to merge 1 commit intomasterfrom
fix-null-pointer-systemd-sonic-generator
Closed

Fix NULL pointer access in systemd-sonic-generator#154
stephenxs wants to merge 1 commit intomasterfrom
fix-null-pointer-systemd-sonic-generator

Conversation

@stephenxs
Copy link
Copy Markdown
Owner

@stephenxs stephenxs commented Jan 8, 2024

Why I did it

Fix NULL pointer access when there is no WantedBy or RequiredBy in the first line.

Signed-off-by: Stephen Sun stephens@nvidia.com

Work item tracking
  • Microsoft ADO (number only):

How I did it

How to verify it

Manual and unit test.

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

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

Tested branch (Please provide the tested image version)

Description for the changelog

Link to config_db schema for YANG module changes

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

Signed-off-by: Stephen Sun <stephens@nvidia.com>
@stephenxs stephenxs force-pushed the fix-null-pointer-systemd-sonic-generator branch from 02572d9 to 57f2e49 Compare January 8, 2024 13:46
@stephenxs
Copy link
Copy Markdown
Owner Author

ci 3446 passed

@stephenxs stephenxs closed this Jan 9, 2024
@stephenxs stephenxs deleted the fix-null-pointer-systemd-sonic-generator branch February 26, 2024 06:29
stephenxs pushed a commit that referenced this pull request Sep 24, 2024
…utomatically (sonic-net#20067)

#### Why I did it
src/sonic-host-services
```
* 052a797 - (HEAD -> 202311, origin/202311) Change dependency option to fix buildimage issue. (#110) (10 hours ago) [Feng-msft]
* 6ab6243 - Fix UT test data due to timestamp break. (#117) (10 hours ago) [Feng-msft]
* 7a68e40 - Update ProcessStats query by using API instead of parsing ps command. (#103) (#154) (11 hours ago) [mssonicbld]
```
#### 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.

2 participants