X-Git-Url: https://diplodocus.org/git/nmh/blobdiff_plain/6b6847272dc5d21ca6fcd23bf20e16193b226c0a..17e279e347cc4636357c694b0f9b4c8ecff2addc:/uip/sendsbr.c

diff --git a/uip/sendsbr.c b/uip/sendsbr.c
index ff203a44..2ba52b1e 100644
--- a/uip/sendsbr.c
+++ b/uip/sendsbr.c
@@ -88,7 +88,7 @@ sendsbr (char **vec, int vecp, char *program, char *draft, struct stat *st,
 	break;
 
     case OK:
-	buildvec = argsplit(buildmimeproc, &buildprogram, &i);
+        buildvec = argsplit(buildmimeproc, &buildprogram, &i);
 	buildvec[i++] = "-auto";
 	if (distfile)
 	    buildvec[i++] = "-dist";
@@ -129,10 +129,15 @@ sendsbr (char **vec, int vecp, char *program, char *draft, struct stat *st,
 	 */
 
 	if (auth_svc) {
+#ifdef OAUTH_SUPPORT
 		const char *errmsg;
 		if (setup_oauth_params(vec, nvecsp, auth_svc, &errmsg) != OK) {
 			adios(NULL, errmsg);
 		}
+#else
+                adios(NULL, "send built without OAUTH_SUPPORT, "
+                      "so auth_svc %s is not supported", auth_svc);
+#endif /* OAUTH_SUPPORT */
 	}
 
         /*
@@ -765,7 +770,8 @@ handle_sendfrom(char **vec, int *vecp, char *draft, const char *auth_svc) {
                 }
                 break;
 #else
-                adios(NULL, "sendfrom built without OAUTH_SUPPORT, "
+                NMH_UNUSED(auth_svc);
+                adios(NULL, "send built without OAUTH_SUPPORT, "
                       "so -saslmech xoauth2 is not supported");
 #endif /* OAUTH_SUPPORT */
             }
@@ -815,22 +821,22 @@ setup_oauth_params(char *vec[], int *vecp, const char *auth_svc,
 	    				    sizeof(errbuf)))
 		adios(NULL, "Unable to retrieve oauth profile entries: %s",
 		      errbuf);
-	    
-	    vec[(*vecp)++] = getcpy("-authservice");
+
+	    vec[(*vecp)++] = mh_xstrdup("-authservice");
 	    vec[(*vecp)++] = getcpy(auth_svc);
-	    vec[(*vecp)++] = getcpy("-oauthcredfile");
+	    vec[(*vecp)++] = mh_xstrdup("-oauthcredfile");
 	    vec[(*vecp)++] = getcpy(mh_oauth_cred_fn(auth_svc));
-	    vec[(*vecp)++] = getcpy("-oauthclientid");
+	    vec[(*vecp)++] = mh_xstrdup("-oauthclientid");
 	    vec[(*vecp)++] = getcpy(svc.client_id);
-	    vec[(*vecp)++] = getcpy("-oauthclientsecret");
+	    vec[(*vecp)++] = mh_xstrdup("-oauthclientsecret");
 	    vec[(*vecp)++] = getcpy(svc.client_secret);
-	    vec[(*vecp)++] = getcpy("-oauthauthendpoint");
+	    vec[(*vecp)++] = mh_xstrdup("-oauthauthendpoint");
 	    vec[(*vecp)++] = getcpy(svc.auth_endpoint);
-	    vec[(*vecp)++] = getcpy("-oauthredirect");
+	    vec[(*vecp)++] = mh_xstrdup("-oauthredirect");
 	    vec[(*vecp)++] = getcpy(svc.redirect_uri);
-	    vec[(*vecp)++] = getcpy("-oauthtokenendpoint");
+	    vec[(*vecp)++] = mh_xstrdup("-oauthtokenendpoint");
 	    vec[(*vecp)++] = getcpy(svc.token_endpoint);
-	    vec[(*vecp)++] = getcpy("-oauthscope");
+	    vec[(*vecp)++] = mh_xstrdup("-oauthscope");
 	    vec[(*vecp)++] = getcpy(svc.scope);
         }
     }
@@ -861,7 +867,7 @@ get_from_header_info(const char *filename, const char **addr, const char **host,
         char *hostformat = "%(host{Envelope-From})";
 
         if ((*addr = get_message_header_info (in, addrformat)) == NULL  ||
-            strlen (*addr) == 0) {
+            !**addr) {
             addrformat = distfile == NULL  ?  "%(addr{Sender})"  :  "%(addr{Resent-Sender})";
             hostformat = distfile == NULL  ?  "%(host{Sender})"  :  "%(host{Resent-Sender})";
 
@@ -957,12 +963,11 @@ get_message_header_info(FILE *in, char *format) {
     /* Trim trailing newline, if any. */
     retval = rtrim(charstring_buffer_copy((buffer)));
     charstring_free(buffer);
-    if (strlen (retval) > 0) {
+    if (*retval)
         return retval;
-    } else {
-        free (retval);
-        return NULL;
-    }
+
+    free(retval);
+    return NULL;
 }
 
 
@@ -992,7 +997,7 @@ merge_profile_entry(const char *addr, const char *host, char *vec[], int *vecp)
     }
 
     /* Use argsplit() to do the real work of splitting the args in the profile entry. */
-    if (profile_entry  &&  strlen(profile_entry) > 0) {
+    if (profile_entry && *profile_entry) {
         int profile_vecp;
         char *file;
         char **profile_vec = argsplit(profile_entry, &file, &profile_vecp);