-
Notifications
You must be signed in to change notification settings - Fork 30
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
[TA1487] Passing correct size to getnameinfo() #77
[TA1487] Passing correct size to getnameinfo() #77
Conversation
Signed-off-by: satbir <[email protected]>
lib/libzrepl/data_conn.c
Outdated
NI_NUMERICSERV); | ||
rc = getnameinfo(&in_addr, in_len, hbuf, | ||
sizeof (NI_MAXHOST), sbuf, sizeof (NI_MAXSERV), | ||
NI_NUMERICHOST | NI_NUMERICSERV); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You need to remove sizeof in front of NI_MAXHOST and NI_MAXSERV and the same for kmem_allocs above. Those are integers and not C types.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks ! Fixed this issue. Please have a look.
Signed-off-by: satbir <[email protected]>
Codecov Report
@@ Coverage Diff @@
## zfs-0.7-release #77 +/- ##
===================================================
- Coverage 50.47% 45.83% -4.64%
===================================================
Files 239 239
Lines 76096 76091 -5
===================================================
- Hits 38406 34874 -3532
- Misses 37690 41217 +3527 Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thanks
Signed-off-by: satbir [email protected]