]> git.dolda2000.com Git - doldaconnect.git/commitdiff
Fix a number of misc. errors and warnings.
authorFredrik Tolf <fredrik@dolda2000.com>
Mon, 14 Jul 2025 19:28:09 +0000 (21:28 +0200)
committerFredrik Tolf <fredrik@dolda2000.com>
Mon, 14 Jul 2025 19:28:09 +0000 (21:28 +0200)
daemon/auth-krb5.c
daemon/client.c
daemon/fnet-dc.c
daemon/net.c
daemon/search.c
include/doldaconnect/uilib.h

index 7b88af31ddd6228f152be2e1813a43c239cd7419..15c7a2fdc762a1c92bc65a93b64d0db7ff2bcc31 100644 (file)
@@ -427,37 +427,37 @@ static int opensess(struct authhandle *auth)
            buf = sprintf2("/tmp/krb5cc_dc_%i_XXXXXX", pwent->pw_uid);
            if((fd = mkstemp(buf)) < 0)
            {
-               free(buf);
                flog(LOG_ERR, "could not create temporary file for ccache: %s", strerror(errno));
+               free(buf);
                return(AUTH_ERR);
            }
            close(fd);
            buf2 = sprintf2("FILE:%s", buf);
            if((ret = krb5_cc_resolve(k5context, buf2, &data->ccache)) != 0)
            {
+               flog(LOG_ERR, "could not resolve ccache name \"%s\": %s", buf2, error_message(ret));
                free(buf);
                free(buf2);
-               flog(LOG_ERR, "could not resolve ccache name \"%s\": %s", buf2, error_message(ret));
                return(AUTH_ERR);
            }
            setenv("KRB5CCNAME", buf2, 1);
            free(buf2);
            if((ret = krb5_cc_initialize(k5context, data->ccache, data->ticket->enc_part2->client)) != 0)
            {
-               free(buf);
                flog(LOG_ERR, "could not initialize ccache: %s", error_message(ret));
+               free(buf);
                return(AUTH_ERR);
            }
            if((ret = krb5_cc_store_cred(k5context, data->ccache, data->creds)) != 0)
            {
-               free(buf);
                flog(LOG_ERR, "could not store forwarded TGT into ccache: %s", error_message(ret));
+               free(buf);
                return(AUTH_ERR);
            }
            if(chown(buf, pwent->pw_uid, pwent->pw_gid))
            {
-               free(buf);
                flog(LOG_ERR, "could not chown new ccache to %i:%i: %s", pwent->pw_uid, pwent->pw_gid, strerror(errno));
+               free(buf);
                return(AUTH_ERR);
            }
            free(buf);
index e493a884ce47f449b80ea86ad3e1901dffce9c54..4993871fe200cf510e01f4458c867e91be0fe19e 100644 (file)
@@ -146,7 +146,7 @@ static int conf_share(int argc, wchar_t **argv)
     return(0);
 }
 
-static void dumpsharecache(struct sharecache *node, int l)
+static void __attribute__((unused)) dumpsharecache(struct sharecache *node, int l)
 {
     int i;
     
index 85ba9a0463ea0f19f8d17c561725f08b3b009251..0fb697ebee2707323be7b60d75d3e20bb76cc038 100644 (file)
@@ -1044,11 +1044,9 @@ static void cmd_myinfo(struct socket *sk, struct fnetnode *fn, char *cmd, char *
  * us. */
 static void cmd_forcemove(struct socket *sk, struct fnetnode *fn, char *cmd, char *args)
 {
-    struct dchub *hub;
     struct fnetnode *newfn;
     int freeargs;
     
-    hub = fn->data;
     if(strchr(args, ':') == NULL)
     {
        args = strcpy(smalloc(strlen(args) + 5), args);
@@ -1388,11 +1386,9 @@ static void cmd_revconnecttome(struct socket *sk, struct fnetnode *fn, char *cmd
 
 static void cmd_getnetinfo(struct socket *sk, struct fnetnode *fn, char *cmd, char *args)
 {
-    struct dchub *hub;
     struct fnetnode *node;
     int numhubs;
     
-    hub = fn->data;
     numhubs = 0;
     for(node = fnetnodes; node != NULL; node = node->next)
     {
@@ -1524,9 +1520,7 @@ static void cmd_getpass(struct socket *sk, struct fnetnode *fn, char *cmd, char
 
 static void cmd_logedin(struct socket *sk, struct fnetnode *fn, char *cmd, char *args)
 {
-    struct dchub *hub;
     
-    hub = fn->data;
     fn->regstatus = FNNS_OP;
     hubhandleaction(sk, fn, cmd, args);
 }
index e1eb5dc2afd116fddc9ecffa22491993750116e8..ef7431d2c49ca44db09a03a96b4d6198a9c33b3b 100644 (file)
@@ -348,10 +348,9 @@ static struct socket *mksock(int domain, int type)
 struct socket *wrapsock(int fd)
 {
     struct socket *sk;
-    struct ufd *ufd;
     
     sk = sockpair(0);
-    ufd = mkufd(fd, UFD_PIPE, sk->back);
+    mkufd(fd, UFD_PIPE, sk->back);
     sksetstate(sk, SOCK_EST);
     fcntl(fd, F_SETFL, fcntl(fd, F_GETFL) | O_NONBLOCK);
     return(sk);
index 633e07ee7fd15b7e03e8c9efd63513e5042180c8..98c4613aa7302a1375313613bb0d1a77f22e930c 100644 (file)
@@ -407,7 +407,7 @@ static struct reinfo analyzere(wchar_t *re, wchar_t **endret, wchar_t endc)
        {
            if(ret.begstr != NULL)
            {
-               for(i = 0; (sinf.begstr[i] != L'\0') && (ret.begstr != L'\0') && (ret.begstr[i] == sinf.begstr[i]); i++);
+               for(i = 0; (sinf.begstr[i] != L'\0') && (ret.begstr[i] != L'\0') && (ret.begstr[i] == sinf.begstr[i]); i++);
                if(i == 0) {
                    free(ret.begstr);
                    ret.begstr = NULL;
index 397c49e5ffdd25ad8a7116cad88a8aef6f7e1791..5b999df89a2ee2ce23db05893889bb5a7cae9161 100644 (file)
@@ -41,7 +41,7 @@ struct dc_intresp
     } *argv;
 };
 
-char *dc_srv_local;
+extern char *dc_srv_local;
 
 int dc_init(void);
 void dc_cleanup(void);