diff options
author | Diego Elio Pettenò <flameeyes@flameeyes.eu> | 2013-02-23 17:18:35 -0800 |
---|---|---|
committer | Diego Elio Pettenò <flameeyes@flameeyes.eu> | 2013-02-23 17:18:35 -0800 |
commit | 7ac103408578ed8ed2ab01e9633021473245913c (patch) | |
tree | b86b583ba42bd42a316d6ffd454ad3d7b80a5208 | |
parent | c61a218efb2a0ec4f493bcc9fa735306f779ea64 (diff) | |
download | libpam-radius-auth-7ac103408578ed8ed2ab01e9633021473245913c.tar.gz libpam-radius-auth-7ac103408578ed8ed2ab01e9633021473245913c.zip |
Fix warnings from compiler.
userinfo is not a pointer-to-pointer, but just a simple pointer.
recvfrom() accepts a generic pointer, rather than a char one, and
requires a socklen_t argument, not an int one.
-rw-r--r-- | pam_radius_auth.c | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/pam_radius_auth.c b/pam_radius_auth.c index abea995..11bec48 100644 --- a/pam_radius_auth.c +++ b/pam_radius_auth.c @@ -766,7 +766,8 @@ static int talk_radius(radius_conf_t *conf, AUTH_HDR *request, AUTH_HDR *response, char *password, char *old_password, int tries) { - int salen, total_length; + socklen_t salen; + int total_length; fd_set set; struct timeval tv; time_t now, end; @@ -881,7 +882,7 @@ send: } else if (FD_ISSET(conf->sockfd, &set)) { /* try to receive some data */ - if ((total_length = recvfrom(conf->sockfd, (char *) response, + if ((total_length = recvfrom(conf->sockfd, (void *) response, BUFFER_SIZE, 0, &saremote, &salen)) < 0) { _pam_log(LOG_ERR, "error reading RADIUS packet from server %s: %s", @@ -1062,7 +1063,7 @@ PAM_EXTERN int pam_sm_authenticate(pam_handle_t *pamh,int flags,int argc,CONST char **argv) { CONST char *user; - CONST char **userinfo; + CONST char *userinfo; char *password = NULL; CONST char *rhost; char *resp2challenge = NULL; |