trying to get HEAD building again. If you want the code
[kai/samba.git] / source3 / client / client.c
index c3d1fbc7ecca5d4040e0169fb7122ccb607f3aa3..d9c3a7aa1b197b6c05628d247150cb27ccae434a 100644 (file)
@@ -1,9 +1,9 @@
 /* 
-   Unix SMB/Netbios implementation.
-   Version 3.0.
+   Unix SMB/CIFS implementation.
    SMB client
    Copyright (C) Andrew Tridgell 1994-1998
-   Copyright (C) Simo Sorce 2001
+   Copyright (C) Simo Sorce 2001-2002
+   Copyright (C) Jelmer Vernooij 2003
    
    This program is free software; you can redistribute it and/or modify
    it under the terms of the GNU General Public License as published by
 #define NO_SYSLOG
 
 #include "includes.h"
-
+#include "../client/client_proto.h"
 #ifndef REGISTER
 #define REGISTER 0
 #endif
 
-const char prog_name[] = "smbclient";
-
 struct cli_state *cli;
 extern BOOL in_client;
 static int port = 0;
 pstring cur_dir = "\\";
-pstring cd_path = "";
+static pstring cd_path = "";
 static pstring service;
 static pstring desthost;
-extern pstring global_myname;
-static pstring password;
 static pstring username;
-static pstring workgroup;
-static char *cmdstr;
+static pstring password;
+static BOOL use_kerberos;
 static BOOL got_pass;
+static char *cmdstr = NULL;
+
 static int io_bufsize = 64512;
-static BOOL use_kerberos;
 
 static int name_type = 0x20;
 static int max_protocol = PROTOCOL_NT1;
-extern pstring user_socket_options;
 
 static int process_tok(fstring tok);
 static int cmd_help(void);
@@ -61,11 +57,9 @@ static int cmd_help(void);
 #define FID_UNUSED (0xFFFF)
 
 time_t newer_than = 0;
-int archive_level = 0;
-
-extern pstring debugf;
+static int archive_level = 0;
 
-BOOL translation = False;
+static BOOL translation = False;
 
 static BOOL have_ip;
 
@@ -76,36 +70,37 @@ extern BOOL tar_reset;
 /* clitar bits end */
  
 
-mode_t myumask = 0755;
-
-BOOL prompt = True;
+static BOOL prompt = True;
 
-int printmode = 1;
+static int printmode = 1;
 
 static BOOL recurse = False;
 BOOL lowercase = False;
 
-struct in_addr dest_ip;
+static struct in_addr dest_ip;
 
 #define SEPARATORS " \t\n\r"
 
-BOOL abort_mget = True;
+static BOOL abort_mget = True;
 
-pstring fileselection = "";
+static pstring fileselection = "";
 
 extern file_info def_finfo;
 
 /* timing globals */
-int get_total_size = 0;
-int get_total_time_ms = 0;
-int put_total_size = 0;
-int put_total_time_ms = 0;
+SMB_BIG_UINT get_total_size = 0;
+unsigned int get_total_time_ms = 0;
+static SMB_BIG_UINT put_total_size = 0;
+static unsigned int put_total_time_ms = 0;
 
 /* totals globals */
 static double dir_total;
 
 #define USENMB
 
+/* some forward declarations */
+static struct cli_state *do_connect(const char *server, const char *share);
+
 /****************************************************************************
 write to a local file with CR/LF->LF translation if appropriate. return the 
 number taken from the buffer. This may not equal the number written.
@@ -306,7 +301,7 @@ static BOOL do_this_one(file_info *finfo)
 
        if (*fileselection && 
            !mask_match(finfo->name,fileselection,False)) {
-               DEBUG(3,("match_match %s failed\n", finfo->name));
+               DEBUG(3,("mask_match %s failed\n", finfo->name));
                return False;
        }
 
@@ -648,22 +643,23 @@ static int cmd_du(void)
 /****************************************************************************
   get a file from rname to lname
   ****************************************************************************/
-static int do_get(char *rname,char *lname)
+static int do_get(char *rname, char *lname, BOOL reget)
 {  
-       int handle=0,fnum;
+       int handle = 0, fnum;
        BOOL newhandle = False;
        char *data;
        struct timeval tp_start;
        int read_size = io_bufsize;
        uint16 attr;
        size_t size;
+       off_t start = 0;
        off_t nread = 0;
        int rc = 0;
 
        GetTimeOfDay(&tp_start);
 
        if (lowercase) {
-               strlower(lname);
+               strlower_m(lname);
        }
 
        fnum = cli_open(cli, rname, O_RDONLY, DENY_NONE);
@@ -676,7 +672,18 @@ static int do_get(char *rname,char *lname)
        if(!strcmp(lname,"-")) {
                handle = fileno(stdout);
        } else {
-               handle = sys_open(lname,O_WRONLY|O_CREAT|O_TRUNC,0644);
+               if (reget) {
+                       handle = sys_open(lname, O_WRONLY|O_CREAT, 0644);
+                       if (handle >= 0) {
+                               start = sys_lseek(handle, 0, SEEK_END);
+                               if (start == -1) {
+                                       d_printf("Error seeking local file\n");
+                                       return 1;
+                               }
+                       }
+               } else {
+                       handle = sys_open(lname, O_WRONLY|O_CREAT|O_TRUNC, 0644);
+               }
                newhandle = True;
        }
        if (handle < 0) {
@@ -694,7 +701,7 @@ static int do_get(char *rname,char *lname)
        }
 
        DEBUG(2,("getting file %s of size %.0f as %s ", 
-                lname, (double)size, lname));
+                rname, (double)size, lname));
 
        if(!(data = (char *)malloc(read_size))) { 
                d_printf("malloc fail for size %d\n", read_size);
@@ -703,7 +710,7 @@ static int do_get(char *rname,char *lname)
        }
 
        while (1) {
-               int n = cli_read(cli, fnum, data, nread, read_size);
+               int n = cli_read(cli, fnum, data, nread + start, read_size);
 
                if (n <= 0) break;
  
@@ -716,7 +723,7 @@ static int do_get(char *rname,char *lname)
                nread += n;
        }
 
-       if (nread < size) {
+       if (nread + start < size) {
                DEBUG (0, ("Short read when getting file %s. Only got %ld bytes.\n",
                            rname, (long)nread));
 
@@ -781,7 +788,7 @@ static int cmd_get(void)
        
        next_token_nr(NULL,lname,NULL,sizeof(lname));
        
-       return do_get(rname, lname);
+       return do_get(rname, lname, False);
 }
 
 
@@ -815,7 +822,7 @@ static void do_mget(file_info *finfo)
        if (!(finfo->mode & aDIR)) {
                pstrcpy(rname,cur_dir);
                pstrcat(rname,finfo->name);
-               do_get(rname,finfo->name);
+               do_get(rname, finfo->name, False);
                return;
        }
 
@@ -827,7 +834,7 @@ static void do_mget(file_info *finfo)
 
        unix_format(finfo->name);
        if (lowercase)
-               strlower(finfo->name);
+               strlower_m(finfo->name);
        
        if (!directory_exist(finfo->name,NULL) && 
            mkdir(finfo->name,0777) != 0) {
@@ -879,7 +886,7 @@ static int cmd_more(void)
        }
        dos_clean_name(rname);
 
-       rc = do_get(rname,lname);
+       rc = do_get(rname, lname, False);
 
        pager=getenv("PAGER");
 
@@ -1045,19 +1052,31 @@ static int cmd_altname(void)
 /****************************************************************************
   put a single file
   ****************************************************************************/
-static int do_put(char *rname,char *lname)
+static int do_put(char *rname, char *lname, BOOL reput)
 {
        int fnum;
        XFILE *f;
-       int nread=0;
-       char *buf=NULL;
-       int maxwrite=io_bufsize;
+       size_t start = 0;
+       off_t nread = 0;
+       char *buf = NULL;
+       int maxwrite = io_bufsize;
        int rc = 0;
        
        struct timeval tp_start;
        GetTimeOfDay(&tp_start);
 
-       fnum = cli_open(cli, rname, O_RDWR|O_CREAT|O_TRUNC, DENY_NONE);
+       if (reput) {
+               fnum = cli_open(cli, rname, O_RDWR|O_CREAT, DENY_NONE);
+               if (fnum >= 0) {
+                       if (!cli_qfileinfo(cli, fnum, NULL, &start, NULL, NULL, NULL, NULL, NULL) &&
+                           !cli_getattrE(cli, fnum, NULL, &start, NULL, NULL, NULL)) {
+                               d_printf("getattrib: %s\n",cli_errstr(cli));
+                               return 1;
+                       }
+               }
+       } else {
+               fnum = cli_open(cli, rname, O_RDWR|O_CREAT|O_TRUNC, DENY_NONE);
+       }
   
        if (fnum == -1) {
                d_printf("%s opening remote file %s\n",cli_errstr(cli),rname);
@@ -1065,12 +1084,21 @@ static int do_put(char *rname,char *lname)
        }
 
        /* allow files to be piped into smbclient
-          jdblair 24.jun.98 */
+          jdblair 24.jun.98
+
+          Note that in this case this function will exit(0) rather
+          than returning. */
        if (!strcmp(lname, "-")) {
                f = x_stdin;
                /* size of file is not known */
        } else {
                f = x_fopen(lname,O_RDONLY, 0);
+               if (f && reput) {
+                       if (x_tseek(f, start, SEEK_SET) == -1) {
+                               d_printf("Error seeking local file\n");
+                               return 1;
+                       }
+               }
        }
 
        if (!f) {
@@ -1100,7 +1128,7 @@ static int do_put(char *rname,char *lname)
                        break;
                }
 
-               ret = cli_write(cli, fnum, 0, buf, nread, n);
+               ret = cli_write(cli, fnum, 0, buf, nread + start, n);
 
                if (n != ret) {
                        d_printf("Error writing file: %s\n", cli_errstr(cli));
@@ -1119,7 +1147,10 @@ static int do_put(char *rname,char *lname)
        }
 
        
-       x_fclose(f);
+       if (f != x_stdin) {
+               x_fclose(f);
+       }
+
        SAFE_FREE(buf);
 
        {
@@ -1185,7 +1216,7 @@ static int cmd_put(void)
                }
        }
 
-       return do_put(rname,lname);
+       return do_put(rname, lname, False);
 }
 
 /*************************************
@@ -1256,7 +1287,7 @@ static int file_find(struct file_list **list, const char *directory,
         int ret;
         char *path;
        BOOL isdir;
-       char *dname;
+       const char *dname;
 
         dir = opendir(directory);
        if (!dir) return -1;
@@ -1344,7 +1375,7 @@ static int cmd_mput(void)
                                /* if (!recurse) continue; */
                                
                                SAFE_FREE(quest);
-                               asprintf(&quest, "Put directory %s? ", lname);
+                               if (asprintf(&quest, "Put directory %s? ", lname) < 0) break;
                                if (prompt && !yesno(quest)) { /* No */
                                        /* Skip the directory */
                                        lname[strlen(lname)-1] = '/';
@@ -1352,7 +1383,7 @@ static int cmd_mput(void)
                                                break;              
                                } else { /* Yes */
                                        SAFE_FREE(rname);
-                                       asprintf(&rname, "%s%s", cur_dir, lname);
+                                       if(asprintf(&rname, "%s%s", cur_dir, lname) < 0) break;
                                        dos_format(rname);
                                        if (!cli_chkpath(cli, rname) && 
                                            !do_mkdir(rname)) {
@@ -1366,18 +1397,18 @@ static int cmd_mput(void)
                                continue;
                        } else {
                                SAFE_FREE(quest);
-                               asprintf(&quest,"Put file %s? ", lname);
+                               if (asprintf(&quest,"Put file %s? ", lname) < 0) break;
                                if (prompt && !yesno(quest)) /* No */
                                        continue;
                                
                                /* Yes */
                                SAFE_FREE(rname);
-                               asprintf(&rname, "%s%s", cur_dir, lname);
+                               if (asprintf(&rname, "%s%s", cur_dir, lname) < 0) break;
                        }
 
                        dos_format(rname);
 
-                       do_put(rname, lname);
+                       do_put(rname, lname, False);
                }
                free_file_list(file_list);
                SAFE_FREE(quest);
@@ -1449,7 +1480,7 @@ static int cmd_print(void)
                slprintf(rname, sizeof(rname)-1, "stdin-%d", (int)sys_getpid());
        }
 
-       return do_put(rname, lname);
+       return do_put(rname, lname, False);
 }
 
 
@@ -1559,6 +1590,148 @@ static int cmd_rmdir(void)
        return 0;
 }
 
+/****************************************************************************
+ UNIX hardlink.
+****************************************************************************/
+
+static int cmd_link(void)
+{
+       pstring src,dest;
+       fstring buf,buf2;
+  
+       if (!SERVER_HAS_UNIX_CIFS(cli)) {
+               d_printf("Server doesn't support UNIX CIFS calls.\n");
+               return 1;
+       }
+
+       pstrcpy(src,cur_dir);
+       pstrcpy(dest,cur_dir);
+  
+       if (!next_token(NULL,buf,NULL,sizeof(buf)) || 
+           !next_token(NULL,buf2,NULL, sizeof(buf2))) {
+               d_printf("link <src> <dest>\n");
+               return 1;
+       }
+
+       pstrcat(src,buf);
+       pstrcat(dest,buf2);
+
+       if (!cli_unix_hardlink(cli, src, dest)) {
+               d_printf("%s linking files (%s -> %s)\n", cli_errstr(cli), src, dest);
+               return 1;
+       }  
+
+       return 0;
+}
+
+/****************************************************************************
+ UNIX symlink.
+****************************************************************************/
+
+static int cmd_symlink(void)
+{
+       pstring src,dest;
+       fstring buf,buf2;
+  
+       if (!SERVER_HAS_UNIX_CIFS(cli)) {
+               d_printf("Server doesn't support UNIX CIFS calls.\n");
+               return 1;
+       }
+
+       pstrcpy(src,cur_dir);
+       pstrcpy(dest,cur_dir);
+       
+       if (!next_token(NULL,buf,NULL,sizeof(buf)) || 
+           !next_token(NULL,buf2,NULL, sizeof(buf2))) {
+               d_printf("symlink <src> <dest>\n");
+               return 1;
+       }
+
+       pstrcat(src,buf);
+       pstrcat(dest,buf2);
+
+       if (!cli_unix_symlink(cli, src, dest)) {
+               d_printf("%s symlinking files (%s -> %s)\n",
+                       cli_errstr(cli), src, dest);
+               return 1;
+       } 
+
+       return 0;
+}
+
+/****************************************************************************
+ UNIX chmod.
+****************************************************************************/
+
+static int cmd_chmod(void)
+{
+       pstring src;
+       mode_t mode;
+       fstring buf, buf2;
+  
+       if (!SERVER_HAS_UNIX_CIFS(cli)) {
+               d_printf("Server doesn't support UNIX CIFS calls.\n");
+               return 1;
+       }
+
+       pstrcpy(src,cur_dir);
+       
+       if (!next_token(NULL,buf,NULL,sizeof(buf)) || 
+           !next_token(NULL,buf2,NULL, sizeof(buf2))) {
+               d_printf("chmod mode file\n");
+               return 1;
+       }
+
+       mode = (mode_t)strtol(buf, NULL, 8);
+       pstrcat(src,buf2);
+
+       if (!cli_unix_chmod(cli, src, mode)) {
+               d_printf("%s chmod file %s 0%o\n",
+                       cli_errstr(cli), src, (unsigned int)mode);
+               return 1;
+       } 
+
+       return 0;
+}
+
+/****************************************************************************
+ UNIX chown.
+****************************************************************************/
+
+static int cmd_chown(void)
+{
+       pstring src;
+       uid_t uid;
+       gid_t gid;
+       fstring buf, buf2, buf3;
+  
+       if (!SERVER_HAS_UNIX_CIFS(cli)) {
+               d_printf("Server doesn't support UNIX CIFS calls.\n");
+               return 1;
+       }
+
+       pstrcpy(src,cur_dir);
+       
+       if (!next_token(NULL,buf,NULL,sizeof(buf)) || 
+           !next_token(NULL,buf2,NULL, sizeof(buf2)) ||
+           !next_token(NULL,buf3,NULL, sizeof(buf3))) {
+               d_printf("chown uid gid file\n");
+               return 1;
+       }
+
+       uid = (uid_t)atoi(buf);
+       gid = (gid_t)atoi(buf2);
+       pstrcat(src,buf3);
+
+       if (!cli_unix_chown(cli, src, uid, gid)) {
+               d_printf("%s chown file %s uid=%d, gid=%d\n",
+                       cli_errstr(cli), src, (int)uid, (int)gid);
+               return 1;
+       } 
+
+       return 0;
+}
+
 /****************************************************************************
 rename some files
 ****************************************************************************/
@@ -1717,8 +1890,8 @@ static int cmd_printmode(void)
 }
 
 /****************************************************************************
-do the lcd command
-****************************************************************************/
+ do the lcd command
+ ****************************************************************************/
 static int cmd_lcd(void)
 {
        fstring buf;
@@ -1732,8 +1905,70 @@ static int cmd_lcd(void)
 }
 
 /****************************************************************************
-list a share name
-****************************************************************************/
+ get a file restarting at end of local file
+ ****************************************************************************/
+static int cmd_reget(void)
+{
+       pstring local_name;
+       pstring remote_name;
+       char *p;
+
+       pstrcpy(remote_name, cur_dir);
+       pstrcat(remote_name, "\\");
+       
+       p = remote_name + strlen(remote_name);
+       
+       if (!next_token_nr(NULL, p, NULL, sizeof(remote_name) - strlen(remote_name))) {
+               d_printf("reget <filename>\n");
+               return 1;
+       }
+       pstrcpy(local_name, p);
+       dos_clean_name(remote_name);
+       
+       next_token_nr(NULL, local_name, NULL, sizeof(local_name));
+       
+       return do_get(remote_name, local_name, True);
+}
+
+/****************************************************************************
+ put a file restarting at end of local file
+ ****************************************************************************/
+static int cmd_reput(void)
+{
+       pstring local_name;
+       pstring remote_name;
+       fstring buf;
+       char *p = buf;
+       SMB_STRUCT_STAT st;
+       
+       pstrcpy(remote_name, cur_dir);
+       pstrcat(remote_name, "\\");
+  
+       if (!next_token_nr(NULL, p, NULL, sizeof(buf))) {
+               d_printf("reput <filename>\n");
+               return 1;
+       }
+       pstrcpy(local_name, p);
+  
+       if (!file_exist(local_name, &st)) {
+               d_printf("%s does not exist\n", local_name);
+               return 1;
+       }
+
+       if (next_token_nr(NULL, p, NULL, sizeof(buf)))
+               pstrcat(remote_name, p);
+       else
+               pstrcat(remote_name, local_name);
+       
+       dos_clean_name(remote_name);
+
+       return do_put(remote_name, local_name, True);
+}
+
+
+/****************************************************************************
+ list a share name
+ ****************************************************************************/
 static void browse_fn(const char *name, uint32 m, 
                       const char *comment, void *state)
 {
@@ -1752,6 +1987,9 @@ static void browse_fn(const char *name, uint32 m,
           case STYPE_IPC:
             fstrcpy(typestr,"IPC"); break;
         }
+       /* FIXME: If the remote machine returns non-ascii characters
+          in any of these fields, they can corrupt the output.  We
+          should remove them. */
        d_printf("\t%-15.15s%-10.10s%s\n",
                name,typestr,comment);
 }
@@ -1812,18 +2050,22 @@ static BOOL list_servers(char *wk_grp)
  *       field is NULL, and NULL in that field is used in process_tok()
  *       (below) to indicate the end of the list.  crh
  */
-struct
+static struct
 {
-  char *name;
+  const char *name;
   int (*fn)(void);
-  char *description;
+  const char *description;
   char compl_args[2];      /* Completion argument info */
 } commands[] = 
 {
+  {"?",cmd_help,"[command] give help on a command",{COMPL_NONE,COMPL_NONE}},
+  {"altname",cmd_altname,"<file> show alt name",{COMPL_NONE,COMPL_NONE}},
   {"archive",cmd_archive,"<level>\n0=ignore archive bit\n1=only get archive files\n2=only get archive files and reset archive bit\n3=get all files and reset archive bit",{COMPL_NONE,COMPL_NONE}},
   {"blocksize",cmd_block,"blocksize <number> (default 20)",{COMPL_NONE,COMPL_NONE}},
   {"cancel",cmd_cancel,"<jobid> cancel a print queue entry",{COMPL_NONE,COMPL_NONE}},
   {"cd",cmd_cd,"[directory] change/report the remote directory",{COMPL_REMOTE,COMPL_NONE}},
+  {"chmod",cmd_chmod,"<src> <mode> chmod a file using UNIX permission",{COMPL_REMOTE,COMPL_REMOTE}},
+  {"chown",cmd_chown,"<src> <uid> <gid> chown a file using UNIX uids and gids",{COMPL_REMOTE,COMPL_REMOTE}},
   {"del",cmd_del,"<mask> delete all matching files",{COMPL_REMOTE,COMPL_NONE}},
   {"dir",cmd_dir,"<mask> list the contents of the current directory",{COMPL_REMOTE,COMPL_NONE}},
   {"du",cmd_du,"<mask> computes the total size of the current directory",{COMPL_REMOTE,COMPL_NONE}},
@@ -1832,6 +2074,7 @@ struct
   {"help",cmd_help,"[command] give help on a command",{COMPL_NONE,COMPL_NONE}},
   {"history",cmd_history,"displays the command history",{COMPL_NONE,COMPL_NONE}},
   {"lcd",cmd_lcd,"[directory] change/report the local current working directory",{COMPL_LOCAL,COMPL_NONE}},
+  {"link",cmd_link,"<src> <dest> create a UNIX hard link",{COMPL_REMOTE,COMPL_REMOTE}},
   {"lowercase",cmd_lowercase,"toggle lowercasing of filenames for get",{COMPL_NONE,COMPL_NONE}},  
   {"ls",cmd_dir,"<mask> list the contents of the current directory",{COMPL_REMOTE,COMPL_NONE}},
   {"mask",cmd_select,"<mask> mask all filenames against this",{COMPL_REMOTE,COMPL_NONE}},
@@ -1852,17 +2095,20 @@ struct
   {"quit",cmd_quit,"logoff the server",{COMPL_NONE,COMPL_NONE}},
   {"rd",cmd_rmdir,"<directory> remove a directory",{COMPL_NONE,COMPL_NONE}},
   {"recurse",cmd_recurse,"toggle directory recursion for mget and mput",{COMPL_NONE,COMPL_NONE}},  
+  {"reget",cmd_reget,"<remote name> [local name] get a file restarting at end of local file",{COMPL_REMOTE,COMPL_LOCAL}},
   {"rename",cmd_rename,"<src> <dest> rename some files",{COMPL_REMOTE,COMPL_REMOTE}},
+  {"reput",cmd_reput,"<local name> [remote name] put a file restarting at end of remote file",{COMPL_LOCAL,COMPL_REMOTE}},
   {"rm",cmd_del,"<mask> delete all matching files",{COMPL_REMOTE,COMPL_NONE}},
   {"rmdir",cmd_rmdir,"<directory> remove a directory",{COMPL_NONE,COMPL_NONE}},
   {"setmode",cmd_setmode,"filename <setmode string> change modes of file",{COMPL_REMOTE,COMPL_NONE}},
+  {"symlink",cmd_symlink,"<src> <dest> create a UNIX symlink",{COMPL_REMOTE,COMPL_REMOTE}},
   {"tar",cmd_tar,"tar <c|x>[IXFqbgNan] current directory to/from <file name>",{COMPL_NONE,COMPL_NONE}},
   {"tarmode",cmd_tarmode,"<full|inc|reset|noreset> tar's behaviour towards archive bits",{COMPL_NONE,COMPL_NONE}},
   {"translate",cmd_translate,"toggle text translation for printing",{COMPL_NONE,COMPL_NONE}},
-  {"altname",cmd_altname,"<file> show alt name",{COMPL_NONE,COMPL_NONE}},
-  {"?",cmd_help,"[command] give help on a command",{COMPL_NONE,COMPL_NONE}},
+  
+  /* Yes, this must be here, see crh's comment above. */
   {"!",NULL,"run a shell command on the local system",{COMPL_NONE,COMPL_NONE}},
-  {"",NULL,NULL,{COMPL_NONE,COMPL_NONE}}
+  {NULL,NULL,NULL,{COMPL_NONE,COMPL_NONE}}
 };
 
 
@@ -1925,9 +2171,17 @@ process a -c command string
 static int process_command_string(char *cmd)
 {
        pstring line;
-       char *ptr;
+       const char *ptr;
        int rc = 0;
 
+       /* establish the connection if not already */
+       
+       if (!cli) {
+               cli = do_connect(desthost, service);
+               if (!cli)
+                       return 0;
+       }
+       
        while (cmd[0] != '\0')    {
                char *p;
                fstring tok;
@@ -1960,40 +2214,187 @@ static int process_command_string(char *cmd)
        return rc;
 }      
 
-/****************************************************************************
-handle completion of commands for readline
-****************************************************************************/
-static char **completion_fn(char *text, int start, int end)
-{
 #define MAX_COMPLETIONS 100
-       char **matches;
-       int i, count=0;
 
-       /* for words not at the start of the line fallback to filename completion */
-       if (start) return NULL;
+typedef struct {
+       pstring dirmask;
+       char **matches;
+       int count, samelen;
+       const char *text;
+       int len;
+} completion_remote_t;
 
-       matches = (char **)malloc(sizeof(matches[0])*MAX_COMPLETIONS);
-       if (!matches) return NULL;
+static void completion_remote_filter(file_info *f, const char *mask, void *state)
+{
+       completion_remote_t *info = (completion_remote_t *)state;
 
-       matches[count++] = strdup(text);
-       if (!matches[0]) return NULL;
+       if ((info->count < MAX_COMPLETIONS - 1) && (strncmp(info->text, f->name, info->len) == 0) && (strcmp(f->name, ".") != 0) && (strcmp(f->name, "..") != 0)) {
+               if ((info->dirmask[0] == 0) && !(f->mode & aDIR))
+                       info->matches[info->count] = strdup(f->name);
+               else {
+                       pstring tmp;
 
-       for (i=0;commands[i].fn && count < MAX_COMPLETIONS-1;i++) {
-               if (strncmp(text, commands[i].name, strlen(text)) == 0) {
-                       matches[count] = strdup(commands[i].name);
-                       if (!matches[count]) return NULL;
-                       count++;
+                       if (info->dirmask[0] != 0)
+                               pstrcpy(tmp, info->dirmask);
+                       else
+                               tmp[0] = 0;
+                       pstrcat(tmp, f->name);
+                       if (f->mode & aDIR)
+                               pstrcat(tmp, "/");
+                       info->matches[info->count] = strdup(tmp);
                }
+               if (info->matches[info->count] == NULL)
+                       return;
+               if (f->mode & aDIR)
+                       smb_readline_ca_char(0);
+
+               if (info->count == 1)
+                       info->samelen = strlen(info->matches[info->count]);
+               else
+                       while (strncmp(info->matches[info->count], info->matches[info->count-1], info->samelen) != 0)
+                               info->samelen--;
+               info->count++;
        }
+}
 
-       if (count == 2) {
-               SAFE_FREE(matches[0]);
-               matches[0] = strdup(matches[1]);
+static char **remote_completion(const char *text, int len)
+{
+       pstring dirmask;
+       int i;
+       completion_remote_t info = { "", NULL, 1, 0, NULL, 0 };
+
+       /* can't have non-static intialisation on Sun CC, so do it
+          at run time here */
+       info.samelen = len;
+       info.text = text;
+       info.len = len;
+               
+       if (len >= PATH_MAX)
+               return(NULL);
+
+       info.matches = (char **)malloc(sizeof(info.matches[0])*MAX_COMPLETIONS);
+       if (!info.matches) return NULL;
+       info.matches[0] = NULL;
+
+       for (i = len-1; i >= 0; i--)
+               if ((text[i] == '/') || (text[i] == '\\'))
+                       break;
+       info.text = text+i+1;
+       info.samelen = info.len = len-i-1;
+
+       if (i > 0) {
+               strncpy(info.dirmask, text, i+1);
+               info.dirmask[i+1] = 0;
+               snprintf(dirmask, sizeof(dirmask), "%s%*s*", cur_dir, i-1, text);
+       } else
+               snprintf(dirmask, sizeof(dirmask), "%s*", cur_dir);
+
+       if (cli_list(cli, dirmask, aDIR | aSYSTEM | aHIDDEN, completion_remote_filter, &info) < 0)
+               goto cleanup;
+
+       if (info.count == 2)
+               info.matches[0] = strdup(info.matches[1]);
+       else {
+               info.matches[0] = malloc(info.samelen+1);
+               if (!info.matches[0])
+                       goto cleanup;
+               strncpy(info.matches[0], info.matches[1], info.samelen);
+               info.matches[0][info.samelen] = 0;
        }
-       matches[count] = NULL;
-       return matches;
+       info.matches[info.count] = NULL;
+       return info.matches;
+
+cleanup:
+       for (i = 0; i < info.count; i++)
+               free(info.matches[i]);
+       free(info.matches);
+       return NULL;
 }
 
+static char **completion_fn(const char *text, int start, int end)
+{
+       smb_readline_ca_char(' ');
+
+       if (start) {
+               const char *buf, *sp;
+               int i;
+               char compl_type;
+
+               buf = smb_readline_get_line_buffer();
+               if (buf == NULL)
+                       return NULL;
+               
+               sp = strchr(buf, ' ');
+               if (sp == NULL)
+                       return NULL;
+               
+               for (i = 0; commands[i].name; i++)
+                       if ((strncmp(commands[i].name, text, sp - buf) == 0) && (commands[i].name[sp - buf] == 0))
+                               break;
+               if (commands[i].name == NULL)
+                       return NULL;
+
+               while (*sp == ' ')
+                       sp++;
+
+               if (sp == (buf + start))
+                       compl_type = commands[i].compl_args[0];
+               else
+                       compl_type = commands[i].compl_args[1];
+
+               if (compl_type == COMPL_REMOTE)
+                       return remote_completion(text, end - start);
+               else /* fall back to local filename completion */
+                       return NULL;
+       } else {
+               char **matches;
+               int i, len, samelen, count=1;
+
+               matches = (char **)malloc(sizeof(matches[0])*MAX_COMPLETIONS);
+               if (!matches) return NULL;
+               matches[0] = NULL;
+
+               len = strlen(text);
+               for (i=0;commands[i].fn && count < MAX_COMPLETIONS-1;i++) {
+                       if (strncmp(text, commands[i].name, len) == 0) {
+                               matches[count] = strdup(commands[i].name);
+                               if (!matches[count])
+                                       goto cleanup;
+                               if (count == 1)
+                                       samelen = strlen(matches[count]);
+                               else
+                                       while (strncmp(matches[count], matches[count-1], samelen) != 0)
+                                               samelen--;
+                               count++;
+                       }
+               }
+
+               switch (count) {
+               case 0: /* should never happen */
+               case 1:
+                       goto cleanup;
+               case 2:
+                       matches[0] = strdup(matches[1]);
+                       break;
+               default:
+                       matches[0] = malloc(samelen+1);
+                       if (!matches[0])
+                               goto cleanup;
+                       strncpy(matches[0], matches[1], samelen);
+                       matches[0][samelen] = 0;
+               }
+               matches[count] = NULL;
+               return matches;
+
+cleanup:
+               while (i >= 0) {
+                       free(matches[i]);
+                       i--;
+               }
+               free(matches);
+               return NULL;
+       }
+}
 
 /****************************************************************************
 make sure we swallow keepalives during idle time
@@ -2012,12 +2413,16 @@ static void readline_callback(void)
        last_t = t;
 
  again:
+
+       if (cli->fd == -1)
+               return;
+
        FD_ZERO(&fds);
        FD_SET(cli->fd,&fds);
 
        timeout.tv_sec = 0;
        timeout.tv_usec = 0;
-       sys_select_intr(cli->fd+1,&fds,&timeout);
+       sys_select_intr(cli->fd+1,&fds,NULL,NULL,&timeout);
                
        /* We deliberately use receive_smb instead of
           client_receive_smb as we want to receive
@@ -2037,7 +2442,7 @@ process commands on stdin
 ****************************************************************************/
 static void process_stdin(void)
 {
-       char *ptr;
+       const char *ptr;
 
        while (1) {
                fstring tok;
@@ -2078,7 +2483,7 @@ static void process_stdin(void)
 /***************************************************** 
 return a connection to a server
 *******************************************************/
-struct cli_state *do_connect(const char *server, const char *share)
+static struct cli_state *do_connect(const char *server, const char *share)
 {
        struct cli_state *c;
        struct nmb_name called, calling;
@@ -2088,7 +2493,7 @@ struct cli_state *do_connect(const char *server, const char *share)
        char *sharename;
        
        /* make a copy so we don't modify the global string 'service' */
-       safe_strcpy(servicename, share, sizeof(servicename)-1);
+       fstrcpy(servicename, share);
        sharename = servicename;
        if (*sharename == '\\') {
                server = sharename+2;
@@ -2102,7 +2507,7 @@ struct cli_state *do_connect(const char *server, const char *share)
        
        zero_ip(&ip);
 
-       make_nmb_name(&calling, global_myname, 0x0);
+       make_nmb_name(&calling, global_myname(), 0x0);
        make_nmb_name(&called , server, name_type);
 
  again:
@@ -2153,28 +2558,26 @@ struct cli_state *do_connect(const char *server, const char *share)
        if (!cli_session_setup(c, username, 
                               password, strlen(password),
                               password, strlen(password),
-                              workgroup)) {
+                              lp_workgroup())) {
                /* if a password was not supplied then try again with a null username */
                if (password[0] || !username[0] || use_kerberos ||
-                   !cli_session_setup(c, "", "", 0, "", 0, workgroup)) { 
+                   !cli_session_setup(c, "", "", 0, "", 0, lp_workgroup())) { 
                        d_printf("session setup failed: %s\n", cli_errstr(c));
+                       if (NT_STATUS_V(cli_nt_error(c)) == 
+                           NT_STATUS_V(NT_STATUS_MORE_PROCESSING_REQUIRED))
+                               d_printf("did you forget to run kinit?\n");
                        cli_shutdown(c);
                        return NULL;
                }
                d_printf("Anonymous login successful\n");
        }
 
-       /*
-        * These next two lines are needed to emulate
-        * old client behaviour for people who have
-        * scripts based on client output.
-        * QUESTION ? Do we want to have a 'client compatibility
-        * mode to turn these on/off ? JRA.
-        */
-
-       if (*c->server_domain || *c->server_os || *c->server_type){
+       if (*c->server_domain) {
                DEBUG(1,("Domain=[%s] OS=[%s] Server=[%s]\n",
                        c->server_domain,c->server_os,c->server_type));
+       } else if (*c->server_os || *c->server_type){
+               DEBUG(1,("OS=[%s] Server=[%s]\n",
+                        c->server_os,c->server_type));
        }               
        
        DEBUG(4,(" session setup ok\n"));
@@ -2216,103 +2619,6 @@ static int process(char *base_directory)
        return rc;
 }
 
-/****************************************************************************
-usage on the program
-****************************************************************************/
-static void usage(char *pname)
-{
-  d_printf("Usage: %s service <password> [options]", pname);
-
-  d_printf("\nVersion %s\n",VERSION);
-  d_printf("\t-s smb.conf           pathname to smb.conf file\n");
-  d_printf("\t-O socket_options     socket options to use\n");
-  d_printf("\t-R name resolve order use these name resolution services only\n");
-  d_printf("\t-M host               send a winpopup message to the host\n");
-  d_printf("\t-i scope              use this NetBIOS scope\n");
-  d_printf("\t-N                    don't ask for a password\n");
-  d_printf("\t-n netbios name.      Use this name as my netbios name\n");
-  d_printf("\t-d debuglevel         set the debuglevel\n");
-  d_printf("\t-P                    connect to service as a printer\n");
-  d_printf("\t-p port               connect to the specified port\n");
-  d_printf("\t-l log basename.      Basename for log/debug files\n");
-  d_printf("\t-h                    Print this help message.\n");
-  d_printf("\t-I dest IP            use this IP to connect to\n");
-  d_printf("\t-E                    write messages to stderr instead of stdout\n");
-  d_printf("\t-k                    use kerberos (active directory) authentication\n");
-  d_printf("\t-U username           set the network username\n");
-  d_printf("\t-L host               get a list of shares available on a host\n");
-  d_printf("\t-t terminal code      terminal i/o code {sjis|euc|jis7|jis8|junet|hex}\n");
-  d_printf("\t-m max protocol       set the max protocol level\n");
-  d_printf("\t-A filename           get the credentials from a file\n");
-  d_printf("\t-W workgroup          set the workgroup name\n");
-  d_printf("\t-T<c|x>IXFqgbNan      command line tar\n");
-  d_printf("\t-D directory          start from directory\n");
-  d_printf("\t-c command string     execute semicolon separated commands\n");
-  d_printf("\t-b xmit/send buffer   changes the transmit/send buffer (default: 65520)\n");
-  d_printf("\n");
-}
-
-
-/****************************************************************************
-get a password from a a file or file descriptor
-exit on failure
-****************************************************************************/
-static void get_password_file(void)
-{
-       int fd = -1;
-       char *p;
-       BOOL close_it = False;
-       pstring spec;
-       char pass[128];
-               
-       if ((p = getenv("PASSWD_FD")) != NULL) {
-               pstrcpy(spec, "descriptor ");
-               pstrcat(spec, p);
-               sscanf(p, "%d", &fd);
-               close_it = False;
-       } else if ((p = getenv("PASSWD_FILE")) != NULL) {
-               fd = sys_open(p, O_RDONLY, 0);
-               pstrcpy(spec, p);
-               if (fd < 0) {
-                       fprintf(stderr, "Error opening PASSWD_FILE %s: %s\n",
-                               spec, strerror(errno));
-                       exit(1);
-               }
-               close_it = True;
-       }
-
-       for(p = pass, *p = '\0'; /* ensure that pass is null-terminated */
-           p && p - pass < sizeof(pass);) {
-               switch (read(fd, p, 1)) {
-               case 1:
-                       if (*p != '\n' && *p != '\0') {
-                               *++p = '\0'; /* advance p, and null-terminate pass */
-                               break;
-                       }
-               case 0:
-                       if (p - pass) {
-                               *p = '\0'; /* null-terminate it, just in case... */
-                               p = NULL; /* then force the loop condition to become false */
-                               break;
-                       } else {
-                               fprintf(stderr, "Error reading password from file %s: %s\n",
-                                       spec, "empty password\n");
-                               exit(1);
-                       }
-                       
-               default:
-                       fprintf(stderr, "Error reading password from file %s: %s\n",
-                               spec, strerror(errno));
-                       exit(1);
-               }
-       }
-       pstrcpy(password, pass);
-       if (close_it)
-               close(fd);
-}      
-
-
-
 /****************************************************************************
 handle a -L query
 ****************************************************************************/
@@ -2323,7 +2629,7 @@ static int do_host_query(char *query_host)
                return 1;
 
        browse_host(True);
-       list_servers(workgroup);
+       list_servers(lp_workgroup());
 
        cli_shutdown(cli);
        
@@ -2337,9 +2643,13 @@ handle a tar operation
 static int do_tar_op(char *base_directory)
 {
        int ret;
-       cli = do_connect(desthost, service);
-       if (!cli)
-               return 1;
+
+       /* do we already have a connection? */
+       if (!cli) {
+               cli = do_connect(desthost, service);    
+               if (!cli)
+                       return 1;
+       }
 
        recurse=True;
 
@@ -2359,16 +2669,21 @@ static int do_message_op(void)
 {
        struct in_addr ip;
        struct nmb_name called, calling;
+       fstring server_name;
+       char name_type_hex[10];
 
-        zero_ip(&ip);
-
-       make_nmb_name(&calling, global_myname, 0x0);
+       make_nmb_name(&calling, global_myname(), 0x0);
        make_nmb_name(&called , desthost, name_type);
 
+       fstrcpy(server_name, desthost);
+       snprintf(name_type_hex, sizeof(name_type_hex), "#%X", name_type);
+       fstrcat(server_name, name_type_hex);
+
         zero_ip(&ip);
        if (have_ip) ip = dest_ip;
 
-       if (!(cli=cli_initialise(NULL)) || (cli_set_port(cli, port) != port) || !cli_connect(cli, desthost, &ip)) {
+       if (!(cli=cli_initialise(NULL)) || (cli_set_port(cli, port) != port) ||
+           !cli_connect(cli, server_name, &ip)) {
                d_printf("Connection to %s failed\n", desthost);
                return 1;
        }
@@ -2386,24 +2701,63 @@ static int do_message_op(void)
 }
 
 
+/**
+ * Process "-L hostname" option.
+ *
+ * We don't actually do anything yet -- we just stash the name in a
+ * global variable and do the query when all options have been read.
+ **/
+static void remember_query_host(const char *arg,
+                               pstring query_host)
+{
+       char *slash;
+       
+       while (*arg == '\\' || *arg == '/')
+               arg++;
+       pstrcpy(query_host, arg);
+       if ((slash = strchr(query_host, '/'))
+           || (slash = strchr(query_host, '\\'))) {
+               *slash = 0;
+       }
+}
+
+
 /****************************************************************************
   main program
 ****************************************************************************/
  int main(int argc,char *argv[])
 {
        fstring base_directory;
-       char *pname = argv[0];
        int opt;
-       extern char *optarg;
-       extern int optind;
-       int old_debug;
        pstring query_host;
        BOOL message = False;
        extern char tar_type;
        pstring term_code;
-       pstring new_name_resolve_order;
+       static const char *new_name_resolve_order = NULL;
+       poptContext pc;
        char *p;
        int rc = 0;
+       struct poptOption long_options[] = {
+               POPT_AUTOHELP
+
+               { "name-resolve", 'R', POPT_ARG_STRING, &new_name_resolve_order, 'R', "Use these name resolution services only", "NAME-RESOLVE-ORDER" },
+               { "message", 'M', POPT_ARG_STRING, NULL, 'M', "Send message", "HOST" },
+               { "ip-address", 'I', POPT_ARG_STRING, NULL, 'I', "Use this IP to connect to", "IP" },
+               { "stderr", 'E', POPT_ARG_NONE, NULL, 'E', "Write messages to stderr instead of stdout" },
+               { "list", 'L', POPT_ARG_STRING, NULL, 'L', "Get a list of shares available on a host", "HOST" },
+               { "terminal", 't', POPT_ARG_STRING, NULL, 't', "Terminal I/O code {sjis|euc|jis7|jis8|junet|hex}", "CODE" },
+               { "max-protocol", 'm', POPT_ARG_STRING, NULL, 'm', "Set the max protocol level", "LEVEL" },
+               { "tar", 'T', POPT_ARG_STRING, NULL, 'T', "Command line tar", "<c|x>IXFqgbNan" },
+               { "directory", 'D', POPT_ARG_STRING, NULL, 'D', "Start from directory", "DIR" },
+               { "command", 'c', POPT_ARG_STRING, &cmdstr, 'c', "Execute semicolon separated commands" }, 
+               { "send-buffer", 'b', POPT_ARG_INT, &io_bufsize, 'b', "Changes the transmit/send buffer", "BYTES" },
+               { "port", 'p', POPT_ARG_INT, &port, 'p', "Port to connect to", "PORT" },
+               POPT_COMMON_SAMBA
+               POPT_COMMON_CONNECTION
+               POPT_COMMON_CREDENTIALS
+               POPT_TABLEEND
+       };
+       
 
 #ifdef KANJI
        pstrcpy(term_code, KANJI);
@@ -2414,302 +2768,104 @@ static int do_message_op(void)
        *query_host = 0;
        *base_directory = 0;
 
-       *new_name_resolve_order = 0;
-
-       DEBUGLEVEL = 2;
-       setup_logging(pname,True);
-
-       /*
-        * If the -E option is given, be careful not to clobber stdout
-        * before processing the options.  28.Feb.99, richard@hacom.nl.
-        * Also pre-parse the -s option to get the service file name.
-        */
-
-       for (opt = 1; opt < argc; opt++) {
-               if (strcmp(argv[opt], "-E") == 0)
-                       dbf = x_stderr;
-               else if(strncmp(argv[opt], "-s", 2) == 0) {
-                       if(argv[opt][2] != '\0')
-                               pstrcpy(dyn_CONFIGFILE, &argv[opt][2]);
-                       else if(argv[opt+1] != NULL) {
-                               /*
-                                * At least one more arg left.
-                                */
-                               pstrcpy(dyn_CONFIGFILE, argv[opt+1]);
-                       } else {
-                               usage(pname);
-                               exit(1);
-                       }
-               }
-       }
-
-       in_client = True;   /* Make sure that we tell lp_load we are */
+       setup_logging(argv[0],True);
 
-       old_debug = DEBUGLEVEL;
        if (!lp_load(dyn_CONFIGFILE,True,False,False)) {
                fprintf(stderr, "%s: Can't load %s - run testparm to debug it\n",
-                       prog_name, dyn_CONFIGFILE);
+                       argv[0], dyn_CONFIGFILE);
        }
-       DEBUGLEVEL = old_debug;
        
-#ifdef WITH_SSL
-       sslutil_init(0);
-#endif
-
-       pstrcpy(workgroup,lp_workgroup());
-
-       load_interfaces();
-       myumask = umask(0);
-       umask(myumask);
-
-       if (getenv("USER")) {
-               pstrcpy(username,getenv("USER"));
-
-               /* modification to support userid%passwd syntax in the USER var
-                  25.Aug.97, jdblair@uab.edu */
-
-               if ((p=strchr_m(username,'%'))) {
-                       *p = 0;
-                       pstrcpy(password,p+1);
-                       got_pass = True;
-                       memset(strchr_m(getenv("USER"),'%')+1,'X',strlen(password));
-               }
-               strupper(username);
-       }
-
-       /* modification to support PASSWD environmental var
-          25.Aug.97, jdblair@uab.edu */
-       if (getenv("PASSWD")) {
-               pstrcpy(password,getenv("PASSWD"));
-               got_pass = True;
-       }
-
-       if (getenv("PASSWD_FD") || getenv("PASSWD_FILE")) {
-               get_password_file();
-               got_pass = True;
-       }
-
-       if (*username == 0 && getenv("LOGNAME")) {
-               pstrcpy(username,getenv("LOGNAME"));
-               strupper(username);
-       }
-
-       if (*username == 0) {
-               pstrcpy(username,"GUEST");
-       }
-
-       if (argc < 2) {
-               usage(pname);
-               exit(1);
-       }
-  
-       if (*argv[1] != '-') {
-               pstrcpy(service,argv[1]);  
-               /* Convert any '/' characters in the service name to '\' characters */
-               string_replace( service, '/','\\');
-               argc--;
-               argv++;
-               
-               if (count_chars(service,'\\') < 3) {
-                       usage(pname);
-                       d_printf("\n%s: Not enough '\\' characters in service\n",service);
-                       exit(1);
-               }
+       pc = poptGetContext("smbclient", argc, (const char **) argv, long_options, 
+                               POPT_CONTEXT_KEEP_FIRST);
+       poptSetOtherOptionHelp(pc, "service <password>");
 
-               if (argc > 1 && (*argv[1] != '-')) {
-                       got_pass = True;
-                       pstrcpy(password,argv[1]);  
-                       memset(argv[1],'X',strlen(argv[1]));
-                       argc--;
-                       argv++;
-               }
-       }
+       in_client = True;   /* Make sure that we tell lp_load we are */
 
-       while ((opt = 
-               getopt(argc, argv,"s:O:R:M:i:Nn:d:Pp:l:hI:EU:L:t:m:W:T:D:c:b:A:k")) != EOF) {
+       while ((opt = poptGetNextOpt(pc)) != -1) {
                switch (opt) {
-               case 's':
-                       pstrcpy(dyn_CONFIGFILE, optarg);
-                       break;
-               case 'O':
-                       pstrcpy(user_socket_options,optarg);
-                       break;  
-               case 'R':
-                       pstrcpy(new_name_resolve_order, optarg);
-                       break;
                case 'M':
-                       name_type = 0x03; /* messages are sent to NetBIOS name type 0x3 */
-                       pstrcpy(desthost,optarg);
-                       message = True;
-                       break;
-               case 'i':
-                       {
-                               extern pstring global_scope;
-                               pstrcpy(global_scope,optarg);
-                               strupper(global_scope);
-                       }
-                       break;
-               case 'N':
-                       got_pass = True;
-                       break;
-               case 'n':
-                       pstrcpy(global_myname,optarg);
-                       break;
-               case 'd':
-                       if (*optarg == 'A')
-                               DEBUGLEVEL = 10000;
-                       else
-                               DEBUGLEVEL = atoi(optarg);
-                       break;
-               case 'P':
-                       /* not needed anymore */
-                       break;
-               case 'p':
-                       port = atoi(optarg);
-                       break;
-               case 'l':
-                       slprintf(debugf,sizeof(debugf)-1, "%s.client",optarg);
-                       break;
-               case 'h':
-                       usage(pname);
-                       exit(0);
-                       break;
+                       /* Messages are sent to NetBIOS name type 0x3
+                        * (Messenger Service).  Make sure we default
+                        * to port 139 instead of port 445. srl,crh
+                        */
+                       name_type = 0x03; 
+                       pstrcpy(desthost,poptGetOptArg(pc));
+                       if( 0 == port ) port = 139;
+                       message = True;
+                       break;
                case 'I':
                        {
-                               dest_ip = *interpret_addr2(optarg);
+                               dest_ip = *interpret_addr2(poptGetOptArg(pc));
                                if (is_zero_ip(dest_ip))
                                        exit(1);
                                have_ip = True;
                        }
                        break;
                case 'E':
-                       display_set_stderr();
                        dbf = x_stderr;
-                       break;
-               case 'U':
-                       {
-                               char *lp;
-                               pstrcpy(username,optarg);
-                               if ((lp=strchr_m(username,'%'))) {
-                                       *lp = 0;
-                                       pstrcpy(password,lp+1);
-                                       got_pass = True;
-                                       memset(strchr_m(optarg,'%')+1,'X',strlen(password));
-                               }
-                       }
-                       break;
-
-               case 'A':
-                       {
-                               XFILE *auth;
-                               fstring buf;
-                               uint16 len = 0;
-                               char *ptr, *val, *param;
-                               
-                               if ((auth=x_fopen(optarg, O_RDONLY, 0)) == NULL)
-                               {
-                                       /* fail if we can't open the credentials file */
-                                       d_printf("ERROR: Unable to open credentials file!\n");
-                                       exit (-1);
-                               }
-                                
-                               while (!x_feof(auth))
-                               {  
-                                       /* get a line from the file */
-                                       if (!x_fgets(buf, sizeof(buf), auth))
-                                               continue;
-                                       len = strlen(buf);
-                                       
-                                       if ((len) && (buf[len-1]=='\n'))
-                                       {
-                                               buf[len-1] = '\0';
-                                               len--;
-                                       }       
-                                       if (len == 0)
-                                               continue;
-                                       
-                                       /* break up the line into parameter & value.
-                                          will need to eat a little whitespace possibly */
-                                       param = buf;
-                                       if (!(ptr = strchr_m (buf, '=')))
-                                               continue;
-                                       val = ptr+1;
-                                       *ptr = '\0';
-                                       
-                                       /* eat leading white space */
-                                       while ((*val!='\0') && ((*val==' ') || (*val=='\t')))
-                                               val++;
-                                       
-                                       if (strwicmp("password", param) == 0)
-                                       {
-                                               pstrcpy(password, val);
-                                               got_pass = True;
-                                       }
-                                       else if (strwicmp("username", param) == 0)
-                                               pstrcpy(username, val);
-                                               
-                                       memset(buf, 0, sizeof(buf));
-                               }
-                               x_fclose(auth);
-                       }
+                       display_set_stderr();
                        break;
 
                case 'L':
-                       p = optarg;
-                       while(*p == '\\' || *p == '/')
-                               p++;
-                       pstrcpy(query_host,p);
+                       remember_query_host(poptGetOptArg(pc), query_host);
                        break;
                case 't':
-                       pstrcpy(term_code, optarg);
+                       pstrcpy(term_code, poptGetOptArg(pc));
                        break;
                case 'm':
-                       max_protocol = interpret_protocol(optarg, max_protocol);
-                       break;
-               case 'W':
-                       pstrcpy(workgroup,optarg);
+                       max_protocol = interpret_protocol(poptGetOptArg(pc), max_protocol);
                        break;
                case 'T':
-                       if (!tar_parseargs(argc, argv, optarg, optind)) {
-                               usage(pname);
+                       if (!tar_parseargs(argc, argv, poptGetOptArg(pc), optind)) {
+                               poptPrintUsage(pc, stderr, 0);
                                exit(1);
                        }
                        break;
                case 'D':
-                       pstrcpy(base_directory,optarg);
+                       fstrcpy(base_directory,poptGetOptArg(pc));
                        break;
-               case 'c':
-                       cmdstr = optarg;
-                       break;
-               case 'b':
-                       io_bufsize = MAX(1, atoi(optarg));
-                       break;
-               case 'k':
-#if HAVE_KRB5
-                       use_kerberos = True;
-                       got_pass = True;
-#else
-                       d_printf("No kerberos support compiled in\n");
-                       exit(1);
-#endif
-                       break;
-               default:
-                       usage(pname);
+               }
+       }
+
+       poptGetArg(pc);
+       
+       load_interfaces();
+
+       if(poptPeekArg(pc)) {
+               pstrcpy(service,poptGetArg(pc));  
+               /* Convert any '/' characters in the service name to '\' characters */
+               string_replace(service, '/','\\');
+
+               if (count_chars(service,'\\') < 3) {
+                       d_printf("\n%s: Not enough '\\' characters in service\n",service);
+                       poptPrintUsage(pc, stderr, 0);
                        exit(1);
                }
        }
 
-       get_myname((*global_myname)?NULL:global_myname);  
+       if (poptPeekArg(pc)) { 
+               cmdline_auth_info.got_pass = True;
+               pstrcpy(cmdline_auth_info.password,poptGetArg(pc));  
+       }
+
+       init_names();
 
-       if(*new_name_resolve_order)
+       if(new_name_resolve_order)
                lp_set_name_resolve_order(new_name_resolve_order);
 
        if (!tar_type && !*query_host && !*service && !message) {
-               usage(pname);
+               poptPrintUsage(pc, stderr, 0);
                exit(1);
        }
 
+       poptFreeContext(pc);
+
+       pstrcpy(username, cmdline_auth_info.username);
+       pstrcpy(password, cmdline_auth_info.password);
+
+       use_kerberos = cmdline_auth_info.use_kerberos;
+       got_pass = cmdline_auth_info.got_pass;
+
        DEBUG( 3, ( "Client started (version %s).\n", VERSION ) );
 
        if (tar_type) {