Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

babel: Added null check after retrieving babel_ifp #16178

Merged
merged 1 commit into from
Jul 30, 2024

Conversation

y-bharath14
Copy link
Contributor

babel: Added null check after retrieving babel_ifp
Asserting the further instructions when the babel interface pointer is NULL

Copy link
Contributor

@mjstapp mjstapp left a comment

Choose a reason for hiding this comment

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

what's the purpose of this- why crash the process one line earlier?

babeld/babel_interface.c Outdated Show resolved Hide resolved
babeld/babel_interface.c Outdated Show resolved Hide resolved
@y-bharath14 y-bharath14 force-pushed the srib-babel-code-refactoring branch 2 times, most recently from 8d948a0 to 269b07f Compare June 7, 2024 05:15
@y-bharath14
Copy link
Contributor Author

what's the purpose of this- why crash the process one line earlier?

Hi Mark Stapp,

Since babel_ip NULL check is handled at most of the places in this file. Just to maintain consistency, adding this check at all the places.
This is just a safe check before accessing babel_ifp pointer.

Thanks & Regards,
Bharath

@ton31337 ton31337 requested a review from mjstapp July 3, 2024 10:06
@riw777
Copy link
Member

riw777 commented Jul 9, 2024

can you rebase to see if that clears the ci failures?

@Jafaral
Copy link
Member

Jafaral commented Jul 9, 2024

Can you please rebase?

@y-bharath14 y-bharath14 force-pushed the srib-babel-code-refactoring branch from 269b07f to b1c426e Compare July 10, 2024 03:57
@y-bharath14
Copy link
Contributor Author

CI:rerun

@y-bharath14
Copy link
Contributor Author

Can you please rebase?

@Jafaral ,
I did rebase and performed force push on this branch.

@y-bharath14
Copy link
Contributor Author

y-bharath14 commented Jul 10, 2024

can you rebase to see if that clears the ci failures?

@riw777,
I did rebase and triggered CI: rerun. Now ci failures are not seen.

Asserting the further instructions when the babel interface pointer is
NULL

Signed-off-by: y-bharath14 <[email protected]>
@y-bharath14 y-bharath14 force-pushed the srib-babel-code-refactoring branch from b1c426e to ae885a7 Compare July 10, 2024 08:16
@y-bharath14
Copy link
Contributor Author

CI:rerun

@mjstapp mjstapp merged commit 292dc38 into FRRouting:master Jul 30, 2024
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants