* assuan-connect.c (assuan_pipe_connect): Require NAME to be set.
authorMarcus Brinkmann <mb@g10code.com>
Mon, 19 Nov 2001 23:44:46 +0000 (23:44 +0000)
committerMarcus Brinkmann <mb@g10code.com>
Mon, 19 Nov 2001 23:44:46 +0000 (23:44 +0000)
Using ARGV[0] automatically is bad mojo.

assuan/assuan-connect.c

index 294a26c..778b3a7 100644 (file)
@@ -31,8 +31,7 @@
 
 /* Connect to a server over a pipe, creating the assuan context and
    returning it in CTX.  The server filename is NAME, the argument
-   vector in ARGV.  If NAME is NULL, the first element in ARGV is
-   used.  */
+   vector in ARGV.  */
 AssuanError
 assuan_pipe_connect (ASSUAN_CONTEXT *ctx, const char *name, char *const argv[])
 {
@@ -42,12 +41,9 @@ assuan_pipe_connect (ASSUAN_CONTEXT *ctx, const char *name, char *const argv[])
   int wp[2];
   int fd[2];
 
-  if (!argv || !argv[0])
+  if (!name || !argv || !argv[0])
     return ASSUAN_General_Error;
 
-  if (!name)
-    name = argv[0];
-
   if (!fixed_signals)
     { 
       struct sigaction act;