+2000-05-29 Ulrich Drepper <drepper@redhat.com>
+
+ * inet/Makefile (CFLAGS-rcmd.c, CFLAGS-rexec.c, CFLAGS-ruserpass.c):
+ Removed.
+ * inet/rcmd.c: Remove __P. Remove unused variables.
+ (iruserfopen): Make first parameter const.
+ (ruserok_sa): Return result of ruserok2_sa.
+ (__checkhost_sa): Correctly iterate through results of getaddrinfo.
+ * inet/rexec.c: Remove unused variables. Other small cleanups.
+ * inet/ruserpass.c: Remove __P. Remove unused variables.
+ (ruserpass): Check results of memory allocation.
+ * include/netdb.h: Add prototype for ruserpass.
+
2000-05-29 Jakub Jelinek <jakub@redhat.com>
* inet/rcmd.c (rresvport_af): Use correct port number.
#include <stdlib.h>
-int __ivaliduser __P ((FILE *, u_int32_t, const char *, const char *));
-static int __validuser2_sa __P ((FILE *, struct sockaddr *, size_t,
- const char *, const char *, const char *));
+int __ivaliduser (FILE *, u_int32_t, const char *, const char *);
+static int __validuser2_sa (FILE *, struct sockaddr *, size_t,
+ const char *, const char *, const char *);
static int ruserok2_sa (struct sockaddr *ra, size_t ralen,
int superuser, const char *ruser,
const char *luser, const char *rhost);
sa_family_t af;
{
char paddr[INET6_ADDRSTRLEN];
- size_t hstbuflen;
- char *tmphstbuf;
struct addrinfo hints, *res, *ai;
struct sockaddr_storage from;
struct pollfd pfd[2];
struct sockaddr_storage ss;
int s;
size_t len;
- uint16_t *sport, i;
+ uint16_t *sport;
switch(family){
case AF_INET:
/* Extremely paranoid file open function. */
static FILE *
-iruserfopen (char *file, uid_t okuser)
+iruserfopen (const char *file, uid_t okuser)
{
struct stat st;
char *cp = NULL;
int superuser;
const char *ruser, *luser;
{
- ruserok2_sa(ra, ralen, superuser, ruser, luser, "-");
+ return ruserok2_sa(ra, ralen, superuser, ruser, luser, "-");
}
/* This is the exported version. */
/* Returns 1 on positive match, 0 on no match, -1 on negative match. */
static int
internal_function
-__checkhost_sa (ra, ralen, lhost, rhost)
- struct sockaddr *ra;
- size_t ralen;
- char *lhost;
- const char *rhost;
+__checkhost_sa (struct sockaddr *ra, size_t ralen, char *lhost,
+ const char *rhost)
{
struct addrinfo hints, *res0, *res;
- int herr;
- int save_errno;
char raddr[INET6_ADDRSTRLEN];
int match;
int negate=1; /* Multiply return with this to get -1 instead of 1 */
- char *user;
/* Check nis netgroup. */
if (strncmp ("+@", lhost, 2) == 0)
hints.ai_family = ra->sa_family;
if (getaddrinfo(lhost, NULL, &hints, &res0) == 0){
/* Spin through ip addresses. */
- for (res=res0; res; res->ai_next){
- if (res->ai_family == ra->sa_family &&
- !memcmp(res->ai_addr, ra, res->ai_addrlen)){
- match = 1;
- break;
- }
- }
- freeaddrinfo(res0);
+ for (res = res0; res; res = res->ai_next)
+ {
+ if (res->ai_family == ra->sa_family
+ && !memcmp(res->ai_addr, ra, res->ai_addrlen))
+ {
+ match = 1;
+ break;
+ }
+ }
+ freeaddrinfo (res0);
}
- return (negate * match);
+ return negate * match;
}
/* Returns 1 on positive match, 0 on no match, -1 on negative match. */
static int
internal_function
-__icheckuser (luser, ruser)
- const char *luser, *ruser;
+__icheckuser (const char *luser, const char *ruser)
{
/*
luser is user entry from .rhosts/hosts.equiv file
ruser is user id on remote host
*/
- char *user;
/* [-+]@netgroup */
if (strncmp ("+@", luser, 2) == 0)
* Returns 1 for blank lines (or only comment lines) and 0 otherwise
*/
static int
-__isempty(p)
- char *p;
+__isempty (char *p)
{
while (*p && isspace (*p)) {
++p;
#include <stdio.h>
#include <netdb.h>
#include <errno.h>
+#include <stdlib.h>
#include <string.h>
#include <unistd.h>
struct addrinfo hints, *res0;
const char *orig_name = name;
const char *orig_pass = pass;
- u_short port;
+ u_short port = 0;
int s, timo = 1, s3;
char c;
- int herr;
- int gai, ok;
+ int gai;
char servbuff[NI_MAXSERV];
snprintf(servbuff, sizeof(servbuff), "%d", rport);
hints.ai_family = af;
hints.ai_socktype = SOCK_STREAM;
hints.ai_flags = AI_CANONNAME;
- if (gai = getaddrinfo(*ahost, servbuff, &hints, &res0)){
+ gai = getaddrinfo(*ahost, servbuff, &hints, &res0);
+ if (gai){
/* XXX: set errno? */
return -1;
}
/* We don't need the memory allocated for the name and the password
in ruserpass anymore. */
if (name != orig_name)
- free (name);
+ free ((char *) name);
if (pass != orig_pass)
- free (pass);
+ free ((char *) pass);
if (__read(s, &c, 1) != 1) {
perror(*ahost);
#include <ctype.h>
#include <err.h>
#include <errno.h>
+#include <netdb.h>
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
/* #include "ftp_var.h" */
-static int token __P((void));
+static int token (void);
static FILE *cfile;
#define DEFAULT 1
int
ruserpass(host, aname, apass)
- char *host, **aname, **apass;
+ const char *host, **aname, **apass;
{
char *hdir, *buf, *tmp;
char myname[1024], *mydomain;
- int t, i, c, usedefault = 0;
+ int t, usedefault = 0;
struct stat stb;
hdir = __secure_getenv("HOME");
while ((t = token()) && t != MACHINE && t != DEFAULT) switch(t) {
case LOGIN:
- if (token())
+ if (token()) {
if (*aname == 0) {
- *aname = malloc((unsigned) strlen(tokval) + 1);
- (void) strcpy(*aname, tokval);
+ char *newp;
+ newp = malloc((unsigned) strlen(tokval) + 1);
+ if (newp == NULL)
+ {
+ warnx(_("out of memory"));
+ goto bad;
+ }
+ *aname = strcpy(newp, tokval);
} else {
if (strcmp(*aname, tokval))
goto next;
}
+ }
break;
case PASSWD:
if (strcmp(*aname, "anonymous") &&
goto bad;
}
if (token() && *apass == 0) {
- *apass = malloc((unsigned) strlen(tokval) + 1);
- (void) strcpy(*apass, tokval);
+ char *newp;
+ newp = malloc((unsigned) strlen(tokval) + 1);
+ if (newp == NULL)
+ {
+ warnx(_("out of memory"));
+ goto bad;
+ }
+ *apass = strcpy(newp, tokval);
}
break;
case ACCOUNT:
projects / platform / upstream / linaro-glibc.git / commitdiff