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

fix: nil pointer #4190

Merged
merged 1 commit into from
Jun 21, 2024
Merged

fix: nil pointer #4190

merged 1 commit into from
Jun 21, 2024

Conversation

zcq98
Copy link
Member

@zcq98 zcq98 commented Jun 19, 2024

Pull Request

What type of this PR

Examples of user facing changes:

  • Bug fixes

Which issue(s) this PR fixes

Fixes #4186

@oilbeater
Copy link
Collaborator

@zcq98 can you add a ut for this?

@zhangzujian
Copy link
Member

在 IPAM Init 过程中判断 subnet 是否 ready,是否更合理?

@zcq98
Copy link
Member Author

zcq98 commented Jun 19, 2024

在 IPAM Init 过程中判断 subnet 是否 ready,是否更合理?

好的,我再去看看代码

@zcq98 zcq98 force-pushed the fix-nil-pointer branch 2 times, most recently from 9fa15f0 to d317e46 Compare June 20, 2024 06:57
pkg/controller/init.go Outdated Show resolved Hide resolved
Signed-off-by: zcq98 <zhaocongqi_yewu@cmss.chinamobile.com>
@bobz965 bobz965 merged commit ccff4ed into kubeovn:master Jun 21, 2024
10 of 11 checks passed
zhangzujian pushed a commit that referenced this pull request Jun 21, 2024
Signed-off-by: zcq98 <zhaocongqi_yewu@cmss.chinamobile.com>
Signed-off-by: zhangzujian <zhangzujian.7@gmail.com>
bobz965 pushed a commit that referenced this pull request Jun 21, 2024
Signed-off-by: zcq98 <zhaocongqi_yewu@cmss.chinamobile.com>
@zcq98 zcq98 deleted the fix-nil-pointer branch June 21, 2024 08:55
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.

[BUG] func GetSubnetIPRangeString nil pointer dereference
4 participants