sync 3.0 into HEAD for the last time
[vlendec/samba-autobuild/.git] / source3 / client / client.c
index 04a30ffb212bb71b581d4221aa2d325fd3940f82..ecece8942ec74e5ff9cac2ce8ee62cf8c09e5226 100644 (file)
@@ -1,8 +1,9 @@
 /* 
-   Unix SMB/Netbios implementation.
-   Version 1.9.
+   Unix SMB/CIFS implementation.
    SMB client
    Copyright (C) Andrew Tridgell 1994-1998
+   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
 
 struct cli_state *cli;
 extern BOOL in_client;
-static int port = SMB_PORT;
+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;
-extern struct in_addr ipzero;
 
 static int name_type = 0x20;
 static int max_protocol = PROTOCOL_NT1;
-extern pstring user_socket_options;
 
 static int process_tok(fstring tok);
-static void cmd_help(void);
+static int cmd_help(void);
 
 /* 30 second timeout on most commands */
 #define CLIENT_TIMEOUT (30*1000)
@@ -58,12 +57,9 @@ static void cmd_help(void);
 #define FID_UNUSED (0xFFFF)
 
 time_t newer_than = 0;
-int archive_level = 0;
+static int archive_level = 0;
 
-extern pstring debugf;
-extern int DEBUGLEVEL;
-
-BOOL translation = False;
+static BOOL translation = False;
 
 static BOOL have_ip;
 
@@ -74,40 +70,42 @@ 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.
+ 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.
 ****************************************************************************/
+
 static int writefile(int f, char *b, int n)
 {
        int i;
@@ -132,20 +130,21 @@ static int writefile(int f, char *b, int n)
 }
 
 /****************************************************************************
 read from a file with LF->CR/LF translation if appropriate. return the 
 number read. read approx n bytes.
Read from a file with LF->CR/LF translation if appropriate. Return the 
+ number read. read approx n bytes.
 ****************************************************************************/
-static int readfile(char *b, int size, int n, FILE *f)
+
+static int readfile(char *b, int n, XFILE *f)
 {
        int i;
        int c;
 
-       if (!translation || (size != 1))
-               return(fread(b,size,n,f));
+       if (!translation)
+               return x_fread(b,1,n,f);
   
        i = 0;
        while (i < (n - 1) && (i < BUFFER_SIZE)) {
-               if ((c = getc(f)) == EOF) {
+               if ((c = x_getc(f)) == EOF) {
                        break;
                }
       
@@ -159,22 +158,22 @@ static int readfile(char *b, int size, int n, FILE *f)
        return(i);
 }
  
-
 /****************************************************************************
-send a message
+ Send a message.
 ****************************************************************************/
+
 static void send_message(void)
 {
        int total_len = 0;
        int grp_id;
 
        if (!cli_message_start(cli, desthost, username, &grp_id)) {
-               DEBUG(0,("message start: %s\n", cli_errstr(cli)));
+               d_printf("message start: %s\n", cli_errstr(cli));
                return;
        }
 
 
-       printf("Connected. Type your message, ending it with a Control-D\n");
+       d_printf("Connected. Type your message, ending it with a Control-D\n");
 
        while (!feof(stdin) && total_len < 1600) {
                int maxlen = MIN(1600 - total_len,127);
@@ -190,15 +189,8 @@ static void send_message(void)
                        msg[l] = c;   
                }
 
-               /*
-                * The message is in UNIX codepage format. Convert to
-                * DOS before sending.
-                */
-
-               unix_to_dos(msg, True);
-
                if (!cli_message_text(cli, msg, l, grp_id)) {
-                       printf("SMBsendtxt failed (%s)\n",cli_errstr(cli));
+                       d_printf("SMBsendtxt failed (%s)\n",cli_errstr(cli));
                        return;
                }      
                
@@ -206,48 +198,51 @@ static void send_message(void)
        }
 
        if (total_len >= 1600)
-               printf("the message was truncated to 1600 bytes\n");
+               d_printf("the message was truncated to 1600 bytes\n");
        else
-               printf("sent %d bytes\n",total_len);
+               d_printf("sent %d bytes\n",total_len);
 
        if (!cli_message_end(cli, grp_id)) {
-               printf("SMBsendend failed (%s)\n",cli_errstr(cli));
+               d_printf("SMBsendend failed (%s)\n",cli_errstr(cli));
                return;
        }      
 }
 
-
-
 /****************************************************************************
-check the space on a device
+ Check the space on a device.
 ****************************************************************************/
-static void do_dskattr(void)
+
+static int do_dskattr(void)
 {
        int total, bsize, avail;
 
        if (!cli_dskattr(cli, &bsize, &total, &avail)) {
-               DEBUG(0,("Error in dskattr: %s\n",cli_errstr(cli))); 
-               return;
+               d_printf("Error in dskattr: %s\n",cli_errstr(cli)); 
+               return 1;
        }
 
-       DEBUG(0,("\n\t\t%d blocks of size %d. %d blocks available\n",
-                total, bsize, avail));
+       d_printf("\n\t\t%d blocks of size %d. %d blocks available\n",
+                total, bsize, avail);
+
+       return 0;
 }
 
 /****************************************************************************
-show cd/pwd
+ Show cd/pwd.
 ****************************************************************************/
-static void cmd_pwd(void)
+
+static int cmd_pwd(void)
 {
-       DEBUG(0,("Current directory is %s",service));
-       DEBUG(0,("%s\n",cur_dir));
+       d_printf("Current directory is %s",service);
+       d_printf("%s\n",cur_dir);
+       return 0;
 }
 
-
 /****************************************************************************
-change directory - inner section
+ Change directory - inner section.
 ****************************************************************************/
-static void do_cd(char *newdir)
+
+static int do_cd(char *newdir)
 {
        char *p = newdir;
        pstring saved_dir;
@@ -272,38 +267,45 @@ static void do_cd(char *newdir)
        
        if (!strequal(cur_dir,"\\")) {
                if (!cli_chkpath(cli, dname)) {
-                       DEBUG(0,("cd %s: %s\n", dname, cli_errstr(cli)));
+                       d_printf("cd %s: %s\n", dname, cli_errstr(cli));
                        pstrcpy(cur_dir,saved_dir);
                }
        }
        
        pstrcpy(cd_path,cur_dir);
+
+       return 0;
 }
 
 /****************************************************************************
-change directory
+ Change directory.
 ****************************************************************************/
-static void cmd_cd(void)
+
+static int cmd_cd(void)
 {
        fstring buf;
+       int rc = 0;
 
-       if (next_token(NULL,buf,NULL,sizeof(buf)))
-               do_cd(buf);
+       if (next_token_nr(NULL,buf,NULL,sizeof(buf)))
+               rc = do_cd(buf);
        else
-               DEBUG(0,("Current directory is %s\n",cur_dir));
-}
+               d_printf("Current directory is %s\n",cur_dir);
 
+       return rc;
+}
 
 /*******************************************************************
-  decide if a file should be operated on
-  ********************************************************************/
+ Decide if a file should be operated on.
+********************************************************************/
+
 static BOOL do_this_one(file_info *finfo)
 {
-       if (finfo->mode & aDIR) return(True);
+       if (finfo->mode & aDIR)
+               return(True);
 
        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;
        }
 
@@ -321,25 +323,26 @@ static BOOL do_this_one(file_info *finfo)
 }
 
 /****************************************************************************
-  display info about a file
-  ****************************************************************************/
+ Display info about a file.
+****************************************************************************/
+
 static void display_finfo(file_info *finfo)
 {
        if (do_this_one(finfo)) {
                time_t t = finfo->mtime; /* the time is assumed to be passed as GMT */
-               DEBUG(0,("  %-30s%7.7s %8.0f  %s",
+               d_printf("  %-30s%7.7s %8.0f  %s",
                         finfo->name,
                         attrib_string(finfo->mode),
                         (double)finfo->size,
-                        asctime(LocalTime(&t))));
+                        asctime(LocalTime(&t)));
                dir_total += finfo->size;
        }
 }
 
-
 /****************************************************************************
-   accumulate size of a file
-  ****************************************************************************/
+ Accumulate size of a file.
+****************************************************************************/
+
 static void do_du(file_info *finfo)
 {
        if (do_this_one(finfo)) {
@@ -356,8 +359,8 @@ static long do_list_queue_end = 0;
 static void (*do_list_fn)(file_info *);
 
 /****************************************************************************
-functions for do_list_queue
-  ****************************************************************************/
+ Functions for do_list_queue.
+****************************************************************************/
 
 /*
  * The do_list_queue is a NUL-separated list of strings stored in a
@@ -370,13 +373,10 @@ functions for do_list_queue
  * Functions check to ensure that do_list_queue is non-NULL before
  * accessing it.
  */
+
 static void reset_do_list_queue(void)
 {
-       if (do_list_queue)
-       {
-               free(do_list_queue);
-       }
-       do_list_queue = 0;
+       SAFE_FREE(do_list_queue);
        do_list_queue_size = 0;
        do_list_queue_start = 0;
        do_list_queue_end = 0;
@@ -388,8 +388,8 @@ static void init_do_list_queue(void)
        do_list_queue_size = 1024;
        do_list_queue = malloc(do_list_queue_size);
        if (do_list_queue == 0) { 
-               DEBUG(0,("malloc fail for size %d\n",
-                        (int)do_list_queue_size));
+               d_printf("malloc fail for size %d\n",
+                        (int)do_list_queue_size);
                reset_do_list_queue();
        } else {
                memset(do_list_queue, 0, do_list_queue_size);
@@ -402,14 +402,11 @@ static void adjust_do_list_queue(void)
         * If the starting point of the queue is more than half way through,
         * move everything toward the beginning.
         */
-       if (do_list_queue && (do_list_queue_start == do_list_queue_end))
-       {
+       if (do_list_queue && (do_list_queue_start == do_list_queue_end)) {
                DEBUG(4,("do_list_queue is empty\n"));
                do_list_queue_start = do_list_queue_end = 0;
                *do_list_queue = '\0';
-       }
-       else if (do_list_queue_start > (do_list_queue_size / 2))
-       {
+       } else if (do_list_queue_start > (do_list_queue_size / 2)) {
                DEBUG(4,("sliding do_list_queue backward\n"));
                memmove(do_list_queue,
                        do_list_queue + do_list_queue_start,
@@ -417,32 +414,30 @@ static void adjust_do_list_queue(void)
                do_list_queue_end -= do_list_queue_start;
                do_list_queue_start = 0;
        }
-          
 }
 
 static void add_to_do_list_queue(const char* entry)
 {
+       char *dlq;
        long new_end = do_list_queue_end + ((long)strlen(entry)) + 1;
-       while (new_end > do_list_queue_size)
-       {
+       while (new_end > do_list_queue_size) {
                do_list_queue_size *= 2;
                DEBUG(4,("enlarging do_list_queue to %d\n",
                         (int)do_list_queue_size));
-               do_list_queue = Realloc(do_list_queue, do_list_queue_size);
-               if (! do_list_queue) {
-                       DEBUG(0,("failure enlarging do_list_queue to %d bytes\n",
-                                (int)do_list_queue_size));
+               dlq = Realloc(do_list_queue, do_list_queue_size);
+               if (! dlq) {
+                       d_printf("failure enlarging do_list_queue to %d bytes\n",
+                                (int)do_list_queue_size);
                        reset_do_list_queue();
-               }
-               else
-               {
+               } else {
+                       do_list_queue = dlq;
                        memset(do_list_queue + do_list_queue_size / 2,
                               0, do_list_queue_size / 2);
                }
        }
-       if (do_list_queue)
-       {
-               pstrcpy(do_list_queue + do_list_queue_end, entry);
+       if (do_list_queue) {
+               safe_strcpy_base(do_list_queue + do_list_queue_end, 
+                                entry, do_list_queue, do_list_queue_size);
                do_list_queue_end = new_end;
                DEBUG(4,("added %s to do_list_queue (start=%d, end=%d)\n",
                         entry, (int)do_list_queue_start, (int)do_list_queue_end));
@@ -456,8 +451,7 @@ static char *do_list_queue_head(void)
 
 static void remove_do_list_queue_head(void)
 {
-       if (do_list_queue_end > do_list_queue_start)
-       {
+       if (do_list_queue_end > do_list_queue_start) {
                do_list_queue_start += strlen(do_list_queue_head()) + 1;
                adjust_do_list_queue();
                DEBUG(4,("removed head of do_list_queue (start=%d, end=%d)\n",
@@ -471,8 +465,9 @@ static int do_list_queue_empty(void)
 }
 
 /****************************************************************************
-a helper for do_list
-  ****************************************************************************/
+ A helper for do_list.
+****************************************************************************/
+
 static void do_list_helper(file_info *f, const char *mask, void *state)
 {
        if (f->mode & aDIR) {
@@ -485,9 +480,15 @@ static void do_list_helper(file_info *f, const char *mask, void *state)
                        pstring mask2;
                        char *p;
 
+                       if (!f->name[0]) {
+                               d_printf("Empty dir name returned. Possible server misconfiguration.\n");
+                               return;
+                       }
+
                        pstrcpy(mask2, mask);
-                       p = strrchr(mask2,'\\');
-                       if (!p) return;
+                       p = strrchr_m(mask2,'\\');
+                       if (!p)
+                               return;
                        p[1] = 0;
                        pstrcat(mask2, f->name);
                        pstrcat(mask2,"\\*");
@@ -501,16 +502,15 @@ static void do_list_helper(file_info *f, const char *mask, void *state)
        }
 }
 
-
 /****************************************************************************
-a wrapper around cli_list that adds recursion
-  ****************************************************************************/
+ A wrapper around cli_list that adds recursion.
+****************************************************************************/
+
 void do_list(const char *mask,uint16 attribute,void (*fn)(file_info *),BOOL rec, BOOL dirs)
 {
        static int in_do_list = 0;
 
-       if (in_do_list && rec)
-       {
+       if (in_do_list && rec) {
                fprintf(stderr, "INTERNAL ERROR: do_list called recursively when the recursive flag is true\n");
                exit(1);
        }
@@ -521,13 +521,11 @@ void do_list(const char *mask,uint16 attribute,void (*fn)(file_info *),BOOL rec,
        do_list_dirs = dirs;
        do_list_fn = fn;
 
-       if (rec)
-       {
+       if (rec) {
                init_do_list_queue();
                add_to_do_list_queue(mask);
                
-               while (! do_list_queue_empty())
-               {
+               while (! do_list_queue_empty()) {
                        /*
                         * Need to copy head so that it doesn't become
                         * invalid inside the call to cli_list.  This
@@ -539,31 +537,25 @@ void do_list(const char *mask,uint16 attribute,void (*fn)(file_info *),BOOL rec,
                        pstrcpy(head, do_list_queue_head());
                        cli_list(cli, head, attribute, do_list_helper, NULL);
                        remove_do_list_queue_head();
-                       if ((! do_list_queue_empty()) && (fn == display_finfo))
-                       {
+                       if ((! do_list_queue_empty()) && (fn == display_finfo)) {
                                char* next_file = do_list_queue_head();
                                char* save_ch = 0;
                                if ((strlen(next_file) >= 2) &&
                                    (next_file[strlen(next_file) - 1] == '*') &&
-                                   (next_file[strlen(next_file) - 2] == '\\'))
-                               {
+                                   (next_file[strlen(next_file) - 2] == '\\')) {
                                        save_ch = next_file +
                                                strlen(next_file) - 2;
                                        *save_ch = '\0';
                                }
-                               DEBUG(0,("\n%s\n",next_file));
-                               if (save_ch)
-                               {
+                               d_printf("\n%s\n",next_file);
+                               if (save_ch) {
                                        *save_ch = '\\';
                                }
                        }
                }
-       }
-       else
-       {
-               if (cli_list(cli, mask, attribute, do_list_helper, NULL) == -1)
-               {
-                       DEBUG(0, ("%s listing %s\n", cli_errstr(cli), mask));
+       } else {
+               if (cli_list(cli, mask, attribute, do_list_helper, NULL) == -1) {
+                       d_printf("%s listing %s\n", cli_errstr(cli), mask);
                }
        }
 
@@ -572,55 +564,59 @@ void do_list(const char *mask,uint16 attribute,void (*fn)(file_info *),BOOL rec,
 }
 
 /****************************************************************************
-  get a directory listing
-  ****************************************************************************/
-static void cmd_dir(void)
+ Get a directory listing.
+****************************************************************************/
+
+static int cmd_dir(void)
 {
        uint16 attribute = aDIR | aSYSTEM | aHIDDEN;
        pstring mask;
        fstring buf;
        char *p=buf;
+       int rc;
        
        dir_total = 0;
        pstrcpy(mask,cur_dir);
        if(mask[strlen(mask)-1]!='\\')
                pstrcat(mask,"\\");
        
-       if (next_token(NULL,buf,NULL,sizeof(buf))) {
+       if (next_token_nr(NULL,buf,NULL,sizeof(buf))) {
                dos_format(p);
                if (*p == '\\')
                        pstrcpy(mask,p);
                else
                        pstrcat(mask,p);
-       }
-       else {
+       } else {
                pstrcat(mask,"*");
        }
 
        do_list(mask, attribute, display_finfo, recurse, True);
 
-       do_dskattr();
+       rc = do_dskattr();
 
        DEBUG(3, ("Total bytes listed: %.0f\n", dir_total));
-}
 
+       return rc;
+}
 
 /****************************************************************************
-  get a directory listing
-  ****************************************************************************/
-static void cmd_du(void)
+ Get a directory listing.
+****************************************************************************/
+
+static int cmd_du(void)
 {
        uint16 attribute = aDIR | aSYSTEM | aHIDDEN;
        pstring mask;
        fstring buf;
        char *p=buf;
+       int rc;
        
        dir_total = 0;
        pstrcpy(mask,cur_dir);
        if(mask[strlen(mask)-1]!='\\')
                pstrcat(mask,"\\");
        
-       if (next_token(NULL,buf,NULL,sizeof(buf))) {
+       if (next_token_nr(NULL,buf,NULL,sizeof(buf))) {
                dos_format(p);
                if (*p == '\\')
                        pstrcpy(mask,p);
@@ -632,48 +628,63 @@ static void cmd_du(void)
 
        do_list(mask, attribute, do_du, recurse, True);
 
-       do_dskattr();
+       rc = do_dskattr();
 
-       DEBUG(0, ("Total number of bytes: %.0f\n", dir_total));
-}
+       d_printf("Total number of bytes: %.0f\n", dir_total);
 
+       return rc;
+}
 
 /****************************************************************************
-  get a file from rname to lname
-  ****************************************************************************/
-static void do_get(char *rname,char *lname)
+ Get a file from rname to 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);
 
        if (fnum == -1) {
-               DEBUG(0,("%s opening remote file %s\n",cli_errstr(cli),rname));
-               return;
+               d_printf("%s opening remote file %s\n",cli_errstr(cli),rname);
+               return 1;
        }
 
        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) {
-               DEBUG(0,("Error opening local file %s\n",lname));
-               return;
+               d_printf("Error opening local file %s\n",lname);
+               return 1;
        }
 
 
@@ -681,41 +692,46 @@ static void do_get(char *rname,char *lname)
                           &attr, &size, NULL, NULL, NULL, NULL, NULL) &&
            !cli_getattrE(cli, fnum, 
                          &attr, &size, NULL, NULL, NULL)) {
-               DEBUG(0,("getattrib: %s\n",cli_errstr(cli)));
-               return;
+               d_printf("getattrib: %s\n",cli_errstr(cli));
+               return 1;
        }
 
        DEBUG(2,("getting file %s of size %.0f as %s ", 
-                lname, (double)size, lname));
+                rname, (double)size, lname));
 
        if(!(data = (char *)malloc(read_size))) { 
-               DEBUG(0,("malloc fail for size %d\n", read_size));
+               d_printf("malloc fail for size %d\n", read_size);
                cli_close(cli, fnum);
-               return;
+               return 1;
        }
 
        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;
+               if (n <= 0)
+                       break;
  
                if (writefile(handle,data, n) != n) {
-                       DEBUG(0,("Error writing local file\n"));
+                       d_printf("Error writing local file\n");
+                       rc = 1;
                        break;
                }
       
                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));
+                           rname, (long)nread));
+
+               rc = 1;
        }
 
-       free(data);
+       SAFE_FREE(data);
        
        if (!cli_close(cli, fnum)) {
-               DEBUG(0,("Error %s closing remote file\n",cli_errstr(cli)));
+               d_printf("Error %s closing remote file\n",cli_errstr(cli));
+               rc = 1;
        }
 
        if (newhandle) {
@@ -741,13 +757,15 @@ static void do_get(char *rname,char *lname)
                         nread / (1.024*this_time + 1.0e-4),
                         get_total_size / (1.024*get_total_time_ms)));
        }
+       
+       return rc;
 }
 
-
 /****************************************************************************
-  get a file
-  ****************************************************************************/
-static void cmd_get(void)
+ Get a file.
+****************************************************************************/
+
+static int cmd_get(void)
 {
        pstring lname;
        pstring rname;
@@ -758,22 +776,22 @@ static void cmd_get(void)
        
        p = rname + strlen(rname);
        
-       if (!next_token(NULL,p,NULL,sizeof(rname)-strlen(rname))) {
-               DEBUG(0,("get <filename>\n"));
-               return;
+       if (!next_token_nr(NULL,p,NULL,sizeof(rname)-strlen(rname))) {
+               d_printf("get <filename>\n");
+               return 1;
        }
        pstrcpy(lname,p);
        dos_clean_name(rname);
        
-       next_token(NULL,lname,NULL,sizeof(lname));
+       next_token_nr(NULL,lname,NULL,sizeof(lname));
        
-       do_get(rname, lname);
+       return do_get(rname, lname, False);
 }
 
-
 /****************************************************************************
-  do a mget operation on one file
-  ****************************************************************************/
+ Do an mget operation on one file.
+****************************************************************************/
+
 static void do_mget(file_info *finfo)
 {
        pstring rname;
@@ -785,7 +803,7 @@ static void do_mget(file_info *finfo)
                return;
 
        if (abort_mget) {
-               DEBUG(0,("mget aborted\n"));
+               d_printf("mget aborted\n");
                return;
        }
 
@@ -796,12 +814,13 @@ static void do_mget(file_info *finfo)
                slprintf(quest,sizeof(pstring)-1,
                         "Get file %s? ",finfo->name);
 
-       if (prompt && !yesno(quest)) return;
+       if (prompt && !yesno(quest))
+               return;
 
        if (!(finfo->mode & aDIR)) {
                pstrcpy(rname,cur_dir);
                pstrcat(rname,finfo->name);
-               do_get(rname,finfo->name);
+               do_get(rname, finfo->name, False);
                return;
        }
 
@@ -813,17 +832,17 @@ 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) {
-               DEBUG(0,("failed to create directory %s\n",finfo->name));
+               d_printf("failed to create directory %s\n",finfo->name);
                pstrcpy(cur_dir,saved_curdir);
                return;
        }
        
        if (chdir(finfo->name) != 0) {
-               DEBUG(0,("failed to chdir to directory %s\n",finfo->name));
+               d_printf("failed to chdir to directory %s\n",finfo->name);
                pstrcpy(cur_dir,saved_curdir);
                return;
        }
@@ -836,44 +855,52 @@ static void do_mget(file_info *finfo)
        pstrcpy(cur_dir,saved_curdir);
 }
 
-
 /****************************************************************************
-view the file using the pager
+ View the file using the pager.
 ****************************************************************************/
-static void cmd_more(void)
+
+static int cmd_more(void)
 {
-       fstring rname,lname,tmpname,pager_cmd;
+       fstring rname,lname,pager_cmd;
        char *pager;
+       int fd;
+       int rc = 0;
 
        fstrcpy(rname,cur_dir);
        fstrcat(rname,"\\");
-       slprintf(tmpname,
-                sizeof(fstring)-1,
-                "%s/smbmore.%d",tmpdir(),(int)sys_getpid());
-       fstrcpy(lname,tmpname);
        
-       if (!next_token(NULL,rname+strlen(rname),NULL,sizeof(rname)-strlen(rname))) {
-               DEBUG(0,("more <filename>\n"));
-               return;
+       slprintf(lname,sizeof(lname)-1, "%s/smbmore.XXXXXX",tmpdir());
+       fd = smb_mkstemp(lname);
+       if (fd == -1) {
+               d_printf("failed to create temporary file for more\n");
+               return 1;
+       }
+       close(fd);
+
+       if (!next_token_nr(NULL,rname+strlen(rname),NULL,sizeof(rname)-strlen(rname))) {
+               d_printf("more <filename>\n");
+               unlink(lname);
+               return 1;
        }
        dos_clean_name(rname);
 
-       do_get(rname,lname);
+       rc = do_get(rname, lname, False);
 
        pager=getenv("PAGER");
 
        slprintf(pager_cmd,sizeof(pager_cmd)-1,
-                "%s %s",(pager? pager:PAGER), tmpname);
+                "%s %s",(pager? pager:PAGER), lname);
        system(pager_cmd);
-       unlink(tmpname);
+       unlink(lname);
+       
+       return rc;
 }
 
-
-
 /****************************************************************************
-do a mget command
+ Do a mget command.
 ****************************************************************************/
-static void cmd_mget(void)
+
+static int cmd_mget(void)
 {
        uint16 attribute = aSYSTEM | aHIDDEN;
        pstring mget_mask;
@@ -887,7 +914,7 @@ static void cmd_mget(void)
        
        abort_mget = False;
 
-       while (next_token(NULL,p,NULL,sizeof(buf))) {
+       while (next_token_nr(NULL,p,NULL,sizeof(buf))) {
                pstrcpy(mget_mask,cur_dir);
                if(mget_mask[strlen(mget_mask)-1]!='\\')
                        pstrcat(mget_mask,"\\");
@@ -906,38 +933,59 @@ static void cmd_mget(void)
                pstrcat(mget_mask,"*");
                do_list(mget_mask, attribute,do_mget,False,True);
        }
+       
+       return 0;
 }
 
-
 /****************************************************************************
-make a directory of name "name"
+ Make a directory of name "name".
 ****************************************************************************/
+
 static BOOL do_mkdir(char *name)
 {
        if (!cli_mkdir(cli, name)) {
-               DEBUG(0,("%s making remote directory %s\n",
-                        cli_errstr(cli),name));
+               d_printf("%s making remote directory %s\n",
+                        cli_errstr(cli),name);
                return(False);
        }
 
        return(True);
 }
 
+/****************************************************************************
+ Show 8.3 name of a file.
+****************************************************************************/
+
+static BOOL do_altname(char *name)
+{
+       fstring altname;
+       if (!NT_STATUS_IS_OK(cli_qpathinfo_alt_name(cli, name, altname))) {
+               d_printf("%s getting alt name for %s\n",
+                        cli_errstr(cli),name);
+               return(False);
+       }
+       d_printf("%s\n", altname);
+
+       return(True);
+}
 
 /****************************************************************************
  Exit client.
 ****************************************************************************/
-static void cmd_quit(void)
+
+static int cmd_quit(void)
 {
        cli_shutdown(cli);
        exit(0);
+       /* NOTREACHED */
+       return 0;
 }
 
-
 /****************************************************************************
-  make a directory
-  ****************************************************************************/
-static void cmd_mkdir(void)
+ Make a directory.
+****************************************************************************/
+
+static int cmd_mkdir(void)
 {
        pstring mask;
        fstring buf;
@@ -945,10 +993,10 @@ static void cmd_mkdir(void)
   
        pstrcpy(mask,cur_dir);
 
-       if (!next_token(NULL,p,NULL,sizeof(buf))) {
+       if (!next_token_nr(NULL,p,NULL,sizeof(buf))) {
                if (!recurse)
-                       DEBUG(0,("mkdir <dirname>\n"));
-               return;
+                       d_printf("mkdir <dirname>\n");
+               return 1;
        }
        pstrcat(mask,p);
 
@@ -958,7 +1006,7 @@ static void cmd_mkdir(void)
                *ddir2 = 0;
                
                pstrcpy(ddir,mask);
-               trim_string(ddir,".",NULL);
+               trim_char(ddir,'.','\0');
                p = strtok(ddir,"/\\");
                while (p) {
                        pstrcat(ddir2,p);
@@ -971,65 +1019,117 @@ static void cmd_mkdir(void)
        } else {
                do_mkdir(mask);
        }
+       
+       return 0;
 }
 
+/****************************************************************************
+ Show alt name.
+****************************************************************************/
+
+static int cmd_altname(void)
+{
+       pstring name;
+       fstring buf;
+       char *p=buf;
+  
+       pstrcpy(name,cur_dir);
+
+       if (!next_token_nr(NULL,p,NULL,sizeof(buf))) {
+               d_printf("altname <file>\n");
+               return 1;
+       }
+       pstrcat(name,p);
+
+       do_altname(name);
+
+       return 0;
+}
 
 /****************************************************************************
-  put a single file
-  ****************************************************************************/
-static void do_put(char *rname,char *lname)
+ Put a single file.
+****************************************************************************/
+
+static int do_put(char *rname, char *lname, BOOL reput)
 {
        int fnum;
-       FILE *f;
-       int nread=0;
-       char *buf=NULL;
-       int maxwrite=io_bufsize;
+       XFILE *f;
+       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) {
-               DEBUG(0,("%s opening remote file %s\n",cli_errstr(cli),rname));
-               return;
+               d_printf("%s opening remote file %s\n",cli_errstr(cli),rname);
+               return 1;
        }
 
        /* 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 = stdin;
+               f = x_stdin;
                /* size of file is not known */
        } else {
-               f = sys_fopen(lname,"r");
+               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) {
-               DEBUG(0,("Error opening local file %s\n",lname));
-               return;
+               d_printf("Error opening local file %s\n",lname);
+               return 1;
        }
-
   
        DEBUG(1,("putting file %s as %s ",lname,
                 rname));
   
        buf = (char *)malloc(maxwrite);
-       while (!feof(f)) {
+       if (!buf) {
+               d_printf("ERROR: Not enough memory!\n");
+               return 1;
+       }
+       while (!x_feof(f)) {
                int n = maxwrite;
                int ret;
 
-               if ((n = readfile(buf,1,n,f)) < 1) {
-                       if((n == 0) && feof(f))
+               if ((n = readfile(buf,n,f)) < 1) {
+                       if((n == 0) && x_feof(f))
                                break; /* Empty local file. */
 
-                       DEBUG(0,("Error reading local file: %s\n", strerror(errno) ));
+                       d_printf("Error reading local file: %s\n", strerror(errno));
+                       rc = 1;
                        break;
                }
 
-               ret = cli_write(cli, fnum, 0, buf, nread, n);
+               ret = cli_write(cli, fnum, 0, buf, nread + start, n);
 
                if (n != ret) {
-                       DEBUG(0,("Error writing file: %s\n", cli_errstr(cli)));
+                       d_printf("Error writing file: %s\n", cli_errstr(cli));
+                       rc = 1;
                        break;
                } 
 
@@ -1037,15 +1137,18 @@ static void do_put(char *rname,char *lname)
        }
 
        if (!cli_close(cli, fnum)) {
-               DEBUG(0,("%s closing remote file %s\n",cli_errstr(cli),rname));
-               fclose(f);
-               if (buf) free(buf);
-               return;
+               d_printf("%s closing remote file %s\n",cli_errstr(cli),rname);
+               x_fclose(f);
+               SAFE_FREE(buf);
+               return 1;
        }
 
        
-       fclose(f);
-       if (buf) free(buf);
+       if (f != x_stdin) {
+               x_fclose(f);
+       }
+
+       SAFE_FREE(buf);
 
        {
                struct timeval tp_end;
@@ -1063,18 +1166,19 @@ static void do_put(char *rname,char *lname)
                         put_total_size / (1.024*put_total_time_ms)));
        }
 
-       if (f == stdin) {
+       if (f == x_stdin) {
                cli_shutdown(cli);
                exit(0);
        }
+       
+       return rc;
 }
 
-
 /****************************************************************************
-  put a file
-  ****************************************************************************/
-static void cmd_put(void)
+ Put a file.
+****************************************************************************/
+
+static int cmd_put(void)
 {
        pstring lname;
        pstring rname;
@@ -1084,13 +1188,13 @@ static void cmd_put(void)
        pstrcpy(rname,cur_dir);
        pstrcat(rname,"\\");
   
-       if (!next_token(NULL,p,NULL,sizeof(buf))) {
-               DEBUG(0,("put <filename>\n"));
-               return;
+       if (!next_token_nr(NULL,p,NULL,sizeof(buf))) {
+               d_printf("put <filename>\n");
+               return 1;
        }
        pstrcpy(lname,p);
   
-       if (next_token(NULL,p,NULL,sizeof(buf)))
+       if (next_token_nr(NULL,p,NULL,sizeof(buf)))
                pstrcat(rname,p);      
        else
                pstrcat(rname,lname);
@@ -1103,171 +1207,273 @@ static void cmd_put(void)
                   jdblair, 24.jun.98 */
                if (!file_exist(lname,&st) &&
                    (strcmp(lname,"-"))) {
-                       DEBUG(0,("%s does not exist\n",lname));
-                       return;
+                       d_printf("%s does not exist\n",lname);
+                       return 1;
                }
        }
 
-       do_put(rname,lname);
+       return do_put(rname, lname, False);
 }
 
+/*************************************
+ File list structure.
+*************************************/
+
+static struct file_list {
+       struct file_list *prev, *next;
+       char *file_path;
+       BOOL isdir;
+} *file_list;
+
+/****************************************************************************
+ Free a file_list structure.
+****************************************************************************/
+
+static void free_file_list (struct file_list * list)
+{
+       struct file_list *tmp;
+       
+       while (list) {
+               tmp = list;
+               DLIST_REMOVE(list, list);
+               SAFE_FREE(tmp->file_path);
+               SAFE_FREE(tmp);
+       }
+}
 
 /****************************************************************************
-  seek in a directory/file list until you get something that doesn't start with
-  the specified name
-  ****************************************************************************/
-static BOOL seek_list(FILE *f,char *name)
+ Seek in a directory/file list until you get something that doesn't start with
+ the specified name.
+****************************************************************************/
+
+static BOOL seek_list(struct file_list *list, char *name)
 {
-       pstring s;
-       while (!feof(f)) {
-               if (!fgets(s,sizeof(s),f)) return(False);
-               trim_string(s,"./","\n");
-               if (strncmp(s,name,strlen(name)) != 0) {
-                       pstrcpy(name,s);
+       while (list) {
+               trim_string(list->file_path,"./","\n");
+               if (strncmp(list->file_path, name, strlen(name)) != 0) {
                        return(True);
                }
+               list = list->next;
        }
       
        return(False);
 }
 
-
 /****************************************************************************
-  set the file selection mask
-  ****************************************************************************/
-static void cmd_select(void)
+ Set the file selection mask.
+****************************************************************************/
+
+static int cmd_select(void)
 {
        pstrcpy(fileselection,"");
-       next_token(NULL,fileselection,NULL,sizeof(fileselection));
+       next_token_nr(NULL,fileselection,NULL,sizeof(fileselection));
+
+       return 0;
 }
 
+/****************************************************************************
+  Recursive file matching function act as find
+  match must be always set to True when calling this function
+****************************************************************************/
+
+static int file_find(struct file_list **list, const char *directory, 
+                     const char *expression, BOOL match)
+{
+       DIR *dir;
+       struct file_list *entry;
+        struct stat statbuf;
+        int ret;
+        char *path;
+       BOOL isdir;
+       const char *dname;
+
+        dir = opendir(directory);
+       if (!dir)
+               return -1;
+       
+        while ((dname = readdirname(dir))) {
+               if (!strcmp("..", dname))
+                       continue;
+               if (!strcmp(".", dname))
+                       continue;
+               
+               if (asprintf(&path, "%s/%s", directory, dname) <= 0) {
+                       continue;
+               }
+
+               isdir = False;
+               if (!match || !gen_fnmatch(expression, dname)) {
+                       if (recurse) {
+                               ret = stat(path, &statbuf);
+                               if (ret == 0) {
+                                       if (S_ISDIR(statbuf.st_mode)) {
+                                               isdir = True;
+                                               ret = file_find(list, path, expression, False);
+                                       }
+                               } else {
+                                       d_printf("file_find: cannot stat file %s\n", path);
+                               }
+                               
+                               if (ret == -1) {
+                                       SAFE_FREE(path);
+                                       closedir(dir);
+                                       return -1;
+                               }
+                       }
+                       entry = (struct file_list *) malloc(sizeof (struct file_list));
+                       if (!entry) {
+                               d_printf("Out of memory in file_find\n");
+                               closedir(dir);
+                               return -1;
+                       }
+                       entry->file_path = path;
+                       entry->isdir = isdir;
+                        DLIST_ADD(*list, entry);
+               } else {
+                       SAFE_FREE(path);
+               }
+        }
+
+       closedir(dir);
+       return 0;
+}
 
 /****************************************************************************
-  mput some files
-  ****************************************************************************/
-static void cmd_mput(void)
+ mput some files.
+****************************************************************************/
+
+static int cmd_mput(void)
 {
-       pstring lname;
-       pstring rname;
        fstring buf;
        char *p=buf;
        
-       while (next_token(NULL,p,NULL,sizeof(buf))) {
-               SMB_STRUCT_STAT st;
-               pstring cmd;
-               pstring tmpname;
-               FILE *f;
-               
-               slprintf(tmpname,sizeof(pstring)-1,
-                        "%s/ls.smb.%d",tmpdir(),(int)sys_getpid());
-               if (recurse)
-                       slprintf(cmd,sizeof(pstring)-1,
-                               "find . -name \"%s\" -print > %s",p,tmpname);
-               else
-                       slprintf(cmd,sizeof(pstring)-1,
-                               "find . -maxdepth 1 -name \"%s\" -print > %s",p,tmpname);
-               system(cmd);
+       while (next_token_nr(NULL,p,NULL,sizeof(buf))) {
+               int ret;
+               struct file_list *temp_list;
+               char *quest, *lname, *rname;
+       
+               file_list = NULL;
 
-               f = sys_fopen(tmpname,"r");
-               if (!f) continue;
+               ret = file_find(&file_list, ".", p, True);
+               if (ret) {
+                       free_file_list(file_list);
+                       continue;
+               }
                
-               while (!feof(f)) {
-                       pstring quest;
+               quest = NULL;
+               lname = NULL;
+               rname = NULL;
+                               
+               for (temp_list = file_list; temp_list; 
+                    temp_list = temp_list->next) {
 
-                       if (!fgets(lname,sizeof(lname),f)) break;
-                       trim_string(lname,"./","\n");
-                       
-               again1:
+                       SAFE_FREE(lname);
+                       if (asprintf(&lname, "%s/", temp_list->file_path) <= 0)
+                               continue;
+                       trim_string(lname, "./", "/");
                        
                        /* check if it's a directory */
-                       if (directory_exist(lname,&st)) {
-                               if (!recurse) continue;
-                               slprintf(quest,sizeof(pstring)-1,
-                                        "Put directory %s? ",lname);
-                               if (prompt && !yesno(quest)) {
-                                       pstrcat(lname,"/");
-                                       if (!seek_list(f,lname))
-                                               break;
-                                       goto again1;                
-                               }
-             
-                               pstrcpy(rname,cur_dir);
-                               pstrcat(rname,lname);
-                               dos_format(rname);
-                               if (!cli_chkpath(cli, rname) && !do_mkdir(rname)) {
-                                       pstrcat(lname,"/");
-                                       if (!seek_list(f,lname))
-                                               break;
-                                       goto again1;
+                       if (temp_list->isdir) {
+                               /* if (!recurse) continue; */
+                               
+                               SAFE_FREE(quest);
+                               if (asprintf(&quest, "Put directory %s? ", lname) < 0) break;
+                               if (prompt && !yesno(quest)) { /* No */
+                                       /* Skip the directory */
+                                       lname[strlen(lname)-1] = '/';
+                                       if (!seek_list(temp_list, lname))
+                                               break;              
+                               } else { /* Yes */
+                                       SAFE_FREE(rname);
+                                       if(asprintf(&rname, "%s%s", cur_dir, lname) < 0) break;
+                                       dos_format(rname);
+                                       if (!cli_chkpath(cli, rname) && 
+                                           !do_mkdir(rname)) {
+                                               DEBUG (0, ("Unable to make dir, skipping..."));
+                                               /* Skip the directory */
+                                               lname[strlen(lname)-1] = '/';
+                                               if (!seek_list(temp_list, lname))
+                                                       break;
+                                       }
                                }
                                continue;
                        } else {
-                               slprintf(quest,sizeof(quest)-1,
-                                        "Put file %s? ",lname);
-                               if (prompt && !yesno(quest)) continue;
+                               SAFE_FREE(quest);
+                               if (asprintf(&quest,"Put file %s? ", lname) < 0) break;
+                               if (prompt && !yesno(quest)) /* No */
+                                       continue;
                                
-                               pstrcpy(rname,cur_dir);
-                               pstrcat(rname,lname);
+                               /* Yes */
+                               SAFE_FREE(rname);
+                               if (asprintf(&rname, "%s%s", cur_dir, lname) < 0) break;
                        }
 
                        dos_format(rname);
 
-                       do_put(rname,lname);
+                       do_put(rname, lname, False);
                }
-               fclose(f);
-               unlink(tmpname);
+               free_file_list(file_list);
+               SAFE_FREE(quest);
+               SAFE_FREE(lname);
+               SAFE_FREE(rname);
        }
-}
 
+       return 0;
+}
 
 /****************************************************************************
-  cancel a print job
-  ****************************************************************************/
-static void do_cancel(int job)
+ Cancel a print job.
+****************************************************************************/
+
+static int do_cancel(int job)
 {
        if (cli_printjob_del(cli, job)) {
-               printf("Job %d cancelled\n",job);
+               d_printf("Job %d cancelled\n",job);
+               return 0;
        } else {
-               printf("Error cancelling job %d : %s\n",job,cli_errstr(cli));
+               d_printf("Error cancelling job %d : %s\n",job,cli_errstr(cli));
+               return 1;
        }
 }
 
-
 /****************************************************************************
-  cancel a print job
-  ****************************************************************************/
-static void cmd_cancel(void)
+ Cancel a print job.
+****************************************************************************/
+
+static int cmd_cancel(void)
 {
        fstring buf;
        int job; 
 
-       if (!next_token(NULL,buf,NULL,sizeof(buf))) {
-               printf("cancel <jobid> ...\n");
-               return;
+       if (!next_token_nr(NULL,buf,NULL,sizeof(buf))) {
+               d_printf("cancel <jobid> ...\n");
+               return 1;
        }
        do {
                job = atoi(buf);
                do_cancel(job);
-       } while (next_token(NULL,buf,NULL,sizeof(buf)));
+       } while (next_token_nr(NULL,buf,NULL,sizeof(buf)));
+       
+       return 0;
 }
 
-
 /****************************************************************************
-  print a file
-  ****************************************************************************/
-static void cmd_print(void)
+ Print a file.
+****************************************************************************/
+
+static int cmd_print(void)
 {
        pstring lname;
        pstring rname;
        char *p;
 
-       if (!next_token(NULL,lname,NULL, sizeof(lname))) {
-               DEBUG(0,("print <filename>\n"));
-               return;
+       if (!next_token_nr(NULL,lname,NULL, sizeof(lname))) {
+               d_printf("print <filename>\n");
+               return 1;
        }
 
        pstrcpy(rname,lname);
-       p = strrchr(rname,'/');
+       p = strrchr_m(rname,'/');
        if (p) {
                slprintf(rname, sizeof(rname)-1, "%s-%d", p+1, (int)sys_getpid());
        }
@@ -1276,29 +1482,33 @@ static void cmd_print(void)
                slprintf(rname, sizeof(rname)-1, "stdin-%d", (int)sys_getpid());
        }
 
-       do_put(rname, lname);
+       return do_put(rname, lname, False);
 }
 
-
 /****************************************************************************
- show a print queue entry
+ Show a print queue entry.
 ****************************************************************************/
+
 static void queue_fn(struct print_job_info *p)
 {
-       DEBUG(0,("%-6d   %-9d    %s\n", (int)p->id, (int)p->size, p->name));
+       d_printf("%-6d   %-9d    %s\n", (int)p->id, (int)p->size, p->name);
 }
 
 /****************************************************************************
- show a print queue
+ Show a print queue.
 ****************************************************************************/
-static void cmd_queue(void)
+
+static int cmd_queue(void)
 {
-       cli_print_queue(cli, queue_fn);  
+       cli_print_queue(cli, queue_fn);
+       
+       return 0;
 }
 
 /****************************************************************************
-delete some files
+ Delete some files.
 ****************************************************************************/
+
 static void do_del(file_info *finfo)
 {
        pstring mask;
@@ -1310,14 +1520,15 @@ static void do_del(file_info *finfo)
                return;
 
        if (!cli_unlink(cli, mask)) {
-               DEBUG(0,("%s deleting remote file %s\n",cli_errstr(cli),mask));
+               d_printf("%s deleting remote file %s\n",cli_errstr(cli),mask);
        }
 }
 
 /****************************************************************************
-delete some files
+ Delete some files.
 ****************************************************************************/
-static void cmd_del(void)
+
+static int cmd_del(void)
 {
        pstring mask;
        fstring buf;
@@ -1328,103 +1539,258 @@ static void cmd_del(void)
        
        pstrcpy(mask,cur_dir);
        
-       if (!next_token(NULL,buf,NULL,sizeof(buf))) {
-               DEBUG(0,("del <filename>\n"));
-               return;
+       if (!next_token_nr(NULL,buf,NULL,sizeof(buf))) {
+               d_printf("del <filename>\n");
+               return 1;
        }
        pstrcat(mask,buf);
 
        do_list(mask, attribute,do_del,False,False);
+       
+       return 0;
 }
 
 /****************************************************************************
 ****************************************************************************/
-static void cmd_open(void)
+
+static int cmd_open(void)
 {
        pstring mask;
        fstring buf;
        
        pstrcpy(mask,cur_dir);
        
-       if (!next_token(NULL,buf,NULL,sizeof(buf))) {
-               DEBUG(0,("del <filename>\n"));
-               return;
+       if (!next_token_nr(NULL,buf,NULL,sizeof(buf))) {
+               d_printf("open <filename>\n");
+               return 1;
        }
        pstrcat(mask,buf);
 
        cli_open(cli, mask, O_RDWR, DENY_ALL);
+
+       return 0;
 }
 
 
 /****************************************************************************
-remove a directory
+ Remove a directory.
 ****************************************************************************/
-static void cmd_rmdir(void)
+
+static int cmd_rmdir(void)
 {
        pstring mask;
        fstring buf;
   
        pstrcpy(mask,cur_dir);
        
-       if (!next_token(NULL,buf,NULL,sizeof(buf))) {
-               DEBUG(0,("rmdir <dirname>\n"));
-               return;
+       if (!next_token_nr(NULL,buf,NULL,sizeof(buf))) {
+               d_printf("rmdir <dirname>\n");
+               return 1;
        }
        pstrcat(mask,buf);
 
        if (!cli_rmdir(cli, mask)) {
-               DEBUG(0,("%s removing remote directory file %s\n",
-                        cli_errstr(cli),mask));
+               d_printf("%s removing remote directory file %s\n",
+                        cli_errstr(cli),mask);
+       }
+       
+       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;
 }
 
 /****************************************************************************
-rename some files
+ UNIX symlink.
 ****************************************************************************/
-static void cmd_rename(void)
+
+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))) {
-               DEBUG(0,("rename <src> <dest>\n"));
-               return;
+               d_printf("symlink <src> <dest>\n");
+               return 1;
        }
 
        pstrcat(src,buf);
        pstrcat(dest,buf2);
 
-       if (!cli_rename(cli, src, dest)) {
-               DEBUG(0,("%s renaming files\n",cli_errstr(cli)));
-               return;
-       }  
+       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;
+}
 
 /****************************************************************************
-toggle the prompt flag
+ Rename some file.
 ****************************************************************************/
-static void cmd_prompt(void)
+
+static int cmd_rename(void)
+{
+       pstring src,dest;
+       fstring buf,buf2;
+  
+       pstrcpy(src,cur_dir);
+       pstrcpy(dest,cur_dir);
+       
+       if (!next_token_nr(NULL,buf,NULL,sizeof(buf)) || 
+           !next_token_nr(NULL,buf2,NULL, sizeof(buf2))) {
+               d_printf("rename <src> <dest>\n");
+               return 1;
+       }
+
+       pstrcat(src,buf);
+       pstrcat(dest,buf2);
+
+       if (!cli_rename(cli, src, dest)) {
+               d_printf("%s renaming files\n",cli_errstr(cli));
+               return 1;
+       }
+       
+       return 0;
+}
+
+/****************************************************************************
+ Toggle the prompt flag.
+****************************************************************************/
+
+static int cmd_prompt(void)
 {
        prompt = !prompt;
        DEBUG(2,("prompting is now %s\n",prompt?"on":"off"));
+       
+       return 1;
 }
 
-
 /****************************************************************************
-set the newer than time
+ Set the newer than time.
 ****************************************************************************/
-static void cmd_newer(void)
+
+static int cmd_newer(void)
 {
        fstring buf;
        BOOL ok;
        SMB_STRUCT_STAT sbuf;
 
-       ok = next_token(NULL,buf,NULL,sizeof(buf));
+       ok = next_token_nr(NULL,buf,NULL,sizeof(buf));
        if (ok && (sys_stat(buf,&sbuf) == 0)) {
                newer_than = sbuf.st_mtime;
                DEBUG(1,("Getting files newer than %s",
@@ -1433,64 +1799,77 @@ static void cmd_newer(void)
                newer_than = 0;
        }
 
-       if (ok && newer_than == 0)
-               DEBUG(0,("Error setting newer-than time\n"));
+       if (ok && newer_than == 0) {
+               d_printf("Error setting newer-than time\n");
+               return 1;
+       }
+
+       return 0;
 }
 
 /****************************************************************************
-set the archive level
+ Set the archive level.
 ****************************************************************************/
-static void cmd_archive(void)
+
+static int cmd_archive(void)
 {
        fstring buf;
 
-       if (next_token(NULL,buf,NULL,sizeof(buf))) {
+       if (next_token_nr(NULL,buf,NULL,sizeof(buf))) {
                archive_level = atoi(buf);
        } else
-               DEBUG(0,("Archive level is %d\n",archive_level));
+               d_printf("Archive level is %d\n",archive_level);
+
+       return 0;
 }
 
 /****************************************************************************
-toggle the lowercaseflag
+ Toggle the lowercaseflag.
 ****************************************************************************/
-static void cmd_lowercase(void)
+
+static int cmd_lowercase(void)
 {
        lowercase = !lowercase;
        DEBUG(2,("filename lowercasing is now %s\n",lowercase?"on":"off"));
-}
-
-
 
+       return 0;
+}
 
 /****************************************************************************
-toggle the recurse flag
+ Toggle the recurse flag.
 ****************************************************************************/
-static void cmd_recurse(void)
+
+static int cmd_recurse(void)
 {
        recurse = !recurse;
        DEBUG(2,("directory recursion is now %s\n",recurse?"on":"off"));
+
+       return 0;
 }
 
 /****************************************************************************
-toggle the translate flag
+ Toggle the translate flag.
 ****************************************************************************/
-static void cmd_translate(void)
+
+static int cmd_translate(void)
 {
        translation = !translation;
        DEBUG(2,("CR/LF<->LF and print text translation now %s\n",
                 translation?"on":"off"));
-}
 
+       return 0;
+}
 
 /****************************************************************************
-do a printmode command
+ Do a printmode command.
 ****************************************************************************/
-static void cmd_printmode(void)
+
+static int cmd_printmode(void)
 {
        fstring buf;
        fstring mode;
 
-       if (next_token(NULL,buf,NULL,sizeof(buf))) {
+       if (next_token_nr(NULL,buf,NULL,sizeof(buf))) {
                if (strequal(buf,"text")) {
                        printmode = 0;      
                } else {
@@ -1501,8 +1880,7 @@ static void cmd_printmode(void)
                }
        }
 
-       switch(printmode)
-               {
+       switch(printmode) {
                case 0: 
                        fstrcpy(mode,"text");
                        break;
@@ -1512,27 +1890,96 @@ static void cmd_printmode(void)
                default: 
                        slprintf(mode,sizeof(mode)-1,"%d",printmode);
                        break;
-               }
+       }
        
        DEBUG(2,("the printmode is now %s\n",mode));
+
+       return 0;
 }
 
 /****************************************************************************
-do the lcd command
-****************************************************************************/
-static void cmd_lcd(void)
+ Do the lcd command.
+ ****************************************************************************/
+
+static int cmd_lcd(void)
 {
        fstring buf;
        pstring d;
        
-       if (next_token(NULL,buf,NULL,sizeof(buf)))
+       if (next_token_nr(NULL,buf,NULL,sizeof(buf)))
                chdir(buf);
        DEBUG(2,("the local directory is now %s\n",sys_getwd(d)));
+
+       return 0;
 }
 
 /****************************************************************************
-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)
 {
@@ -1551,51 +1998,56 @@ static void browse_fn(const char *name, uint32 m,
           case STYPE_IPC:
             fstrcpy(typestr,"IPC"); break;
         }
-
-        printf("\t%-15.15s%-10.10s%s\n",
-               name, typestr,comment);
+       /* 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);
 }
 
-
 /****************************************************************************
-try and browse available connections on a host
+ Try and browse available connections on a host.
 ****************************************************************************/
+
 static BOOL browse_host(BOOL sort)
 {
        int ret;
 
-        printf("\n\tSharename      Type      Comment\n");
-        printf("\t---------      ----      -------\n");
+        d_printf("\n\tSharename      Type      Comment\n");
+        d_printf("\t---------      ----      -------\n");
 
        if((ret = cli_RNetShareEnum(cli, browse_fn, NULL)) == -1)
-               printf("Error returning browse list: %s\n", cli_errstr(cli));
+               d_printf("Error returning browse list: %s\n", cli_errstr(cli));
 
        return (ret != -1);
 }
 
 /****************************************************************************
-list a server name
+ List a server name.
 ****************************************************************************/
+
 static void server_fn(const char *name, uint32 m, 
                       const char *comment, void *state)
 {
-        printf("\t%-16.16s     %s\n", name, comment);
+        d_printf("\t%-16.16s     %s\n", name, comment);
 }
 
 /****************************************************************************
-try and browse available connections on a host
+ Try and browse available connections on a host.
 ****************************************************************************/
+
 static BOOL list_servers(char *wk_grp)
 {
-       if (!cli->server_domain) return False;
-
-        printf("\n\tServer               Comment\n");
-        printf("\t---------            -------\n");
+       if (!cli->server_domain)
+               return False;
+       
+        d_printf("\n\tServer               Comment\n");
+        d_printf("\t---------            -------\n");
 
        cli_NetServerEnum(cli, cli->server_domain, SV_TYPE_ALL, server_fn, NULL);
 
-        printf("\n\tWorkgroup            Master\n");
-        printf("\t---------            -------\n");
+        d_printf("\n\tWorkgroup            Master\n");
+        d_printf("\t---------            -------\n");
 
        cli_NetServerEnum(cli, cli->server_domain, SV_TYPE_DOMAIN_ENUM, server_fn, NULL);
        return True;
@@ -1607,65 +2059,76 @@ static BOOL list_servers(char *wk_grp)
 #define COMPL_REMOTE      1          /* Complete remote filename */
 #define COMPL_LOCAL       2          /* Complete local filename */
 
-/* This defines the commands supported by this client */
-struct
+/* This defines the commands supported by this client.
+ * NOTE: The "!" must be the last one in the list because it's fn pointer
+ *       field is NULL, and NULL in that field is used in process_tok()
+ *       (below) to indicate the end of the list.  crh
+ */
+static struct
 {
-  char *name;
-  void (*fn)(void);
-  char *description;
+  const char *name;
+  int (*fn)(void);
+  const char *description;
   char compl_args[2];      /* Completion argument info */
-} commands[] = 
-{
-  {"ls",cmd_dir,"<mask> list the contents of the current directory",{COMPL_REMOTE,COMPL_NONE}},
+} 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}},
-  {"lcd",cmd_lcd,"[directory] change/report the local current working directory",{COMPL_LOCAL,COMPL_NONE}},
-  {"cd",cmd_cd,"[directory] change/report the remote directory",{COMPL_REMOTE,COMPL_NONE}},
-  {"pwd",cmd_pwd,"show current remote directory (same as 'cd' with no args)",{COMPL_NONE,COMPL_NONE}},
+  {"exit",cmd_quit,"logoff the server",{COMPL_NONE,COMPL_NONE}},
   {"get",cmd_get,"<remote name> [local name] get a file",{COMPL_REMOTE,COMPL_LOCAL}},
+  {"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}},
+  {"md",cmd_mkdir,"<directory> make a directory",{COMPL_NONE,COMPL_NONE}},
   {"mget",cmd_mget,"<mask> get all the matching files",{COMPL_REMOTE,COMPL_NONE}},
-  {"put",cmd_put,"<local name> [remote name] put a file",{COMPL_LOCAL,COMPL_REMOTE}},
-  {"mput",cmd_mput,"<mask> put all matching files",{COMPL_REMOTE,COMPL_NONE}},
-  {"rename",cmd_rename,"<src> <dest> rename some files",{COMPL_REMOTE,COMPL_REMOTE}},
+  {"mkdir",cmd_mkdir,"<directory> make a directory",{COMPL_NONE,COMPL_NONE}},
   {"more",cmd_more,"<remote name> view a remote file with your pager",{COMPL_REMOTE,COMPL_NONE}},  
-  {"mask",cmd_select,"<mask> mask all filenames against this",{COMPL_REMOTE,COMPL_NONE}},
-  {"del",cmd_del,"<mask> delete all matching files",{COMPL_REMOTE,COMPL_NONE}},
+  {"mput",cmd_mput,"<mask> put all matching files",{COMPL_REMOTE,COMPL_NONE}},
+  {"newer",cmd_newer,"<file> only mget files newer than the specified local file",{COMPL_LOCAL,COMPL_NONE}},
   {"open",cmd_open,"<mask> open a file",{COMPL_REMOTE,COMPL_NONE}},
-  {"rm",cmd_del,"<mask> delete all matching files",{COMPL_REMOTE,COMPL_NONE}},
-  {"mkdir",cmd_mkdir,"<directory> make a directory",{COMPL_NONE,COMPL_NONE}},
-  {"md",cmd_mkdir,"<directory> make a directory",{COMPL_NONE,COMPL_NONE}},
-  {"rmdir",cmd_rmdir,"<directory> remove a directory",{COMPL_NONE,COMPL_NONE}},
-  {"rd",cmd_rmdir,"<directory> remove a directory",{COMPL_NONE,COMPL_NONE}},
-  {"prompt",cmd_prompt,"toggle prompting for filenames for mget and mput",{COMPL_NONE,COMPL_NONE}},  
-  {"recurse",cmd_recurse,"toggle directory recursion for mget and mput",{COMPL_NONE,COMPL_NONE}},  
-  {"translate",cmd_translate,"toggle text translation for printing",{COMPL_NONE,COMPL_NONE}},  
-  {"lowercase",cmd_lowercase,"toggle lowercasing of filenames for get",{COMPL_NONE,COMPL_NONE}},  
   {"print",cmd_print,"<file name> print a file",{COMPL_NONE,COMPL_NONE}},
   {"printmode",cmd_printmode,"<graphics or text> set the print mode",{COMPL_NONE,COMPL_NONE}},
+  {"prompt",cmd_prompt,"toggle prompting for filenames for mget and mput",{COMPL_NONE,COMPL_NONE}},  
+  {"put",cmd_put,"<local name> [remote name] put a file",{COMPL_LOCAL,COMPL_REMOTE}},
+  {"pwd",cmd_pwd,"show current remote directory (same as 'cd' with no args)",{COMPL_NONE,COMPL_NONE}},
+  {"q",cmd_quit,"logoff the server",{COMPL_NONE,COMPL_NONE}},
   {"queue",cmd_queue,"show the print queue",{COMPL_NONE,COMPL_NONE}},
-  {"cancel",cmd_cancel,"<jobid> cancel a print queue entry",{COMPL_NONE,COMPL_NONE}},
   {"quit",cmd_quit,"logoff the server",{COMPL_NONE,COMPL_NONE}},
-  {"q",cmd_quit,"logoff the server",{COMPL_NONE,COMPL_NONE}},
-  {"exit",cmd_quit,"logoff the server",{COMPL_NONE,COMPL_NONE}},
-  {"newer",cmd_newer,"<file> only mget files newer than the specified local file",{COMPL_LOCAL,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}},
-  {"tar",cmd_tar,"tar <c|x>[IXFqbgNan] current directory to/from <file name>",{COMPL_NONE,COMPL_NONE}},
-  {"blocksize",cmd_block,"blocksize <number> (default 20)",{COMPL_NONE,COMPL_NONE}},
-  {"tarmode",cmd_tarmode,
-     "<full|inc|reset|noreset> tar's behaviour towards archive bits",{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}},
-  {"help",cmd_help,"[command] give help on a command",{COMPL_NONE,COMPL_NONE}},
-  {"?",cmd_help,"[command] give help on a command",{COMPL_NONE,COMPL_NONE}},
-  {"history",cmd_history,"displays the command history",{COMPL_NONE,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}},
+  
+  /* 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}}
 };
 
-
 /*******************************************************************
-  lookup a command string in the list of commands, including 
-  abbreviations
-  ******************************************************************/
+ Lookup a command string in the list of commands, including 
+ abbreviations.
+******************************************************************/
+
 static int process_tok(fstring tok)
 {
        int i = 0, matches = 0;
@@ -1693,46 +2156,59 @@ static int process_tok(fstring tok)
 }
 
 /****************************************************************************
-help
+ Help.
 ****************************************************************************/
-static void cmd_help(void)
+
+static int cmd_help(void)
 {
        int i=0,j;
        fstring buf;
        
-       if (next_token(NULL,buf,NULL,sizeof(buf))) {
+       if (next_token_nr(NULL,buf,NULL,sizeof(buf))) {
                if ((i = process_tok(buf)) >= 0)
-                       DEBUG(0,("HELP %s:\n\t%s\n\n",commands[i].name,commands[i].description));                   
+                       d_printf("HELP %s:\n\t%s\n\n",commands[i].name,commands[i].description);
        } else {
                while (commands[i].description) {
                        for (j=0; commands[i].description && (j<5); j++) {
-                               DEBUG(0,("%-15s",commands[i].name));
+                               d_printf("%-15s",commands[i].name);
                                i++;
                        }
-                       DEBUG(0,("\n"));
+                       d_printf("\n");
                }
        }
+       return 0;
 }
 
 /****************************************************************************
-process a -c command string
+ Process a -c command string.
 ****************************************************************************/
-static void process_command_string(char *cmd)
+
+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;
                int i;
                
-               if ((p = strchr(cmd, ';')) == 0) {
+               if ((p = strchr_m(cmd, ';')) == 0) {
                        strncpy(line, cmd, 999);
                        line[1000] = '\0';
                        cmd += strlen(cmd);
                } else {
-                       if (p - cmd > 999) p = cmd + 999;
+                       if (p - cmd > 999)
+                               p = cmd + 999;
                        strncpy(line, cmd, p - cmd);
                        line[p - cmd] = '\0';
                        cmd = p + 1;
@@ -1740,56 +2216,206 @@ static void process_command_string(char *cmd)
                
                /* and get the first part of the command */
                ptr = line;
-               if (!next_token(&ptr,tok,NULL,sizeof(tok))) continue;
+               if (!next_token_nr(&ptr,tok,NULL,sizeof(tok))) continue;
                
                if ((i = process_tok(tok)) >= 0) {
-                       commands[i].fn();
+                       rc = commands[i].fn();
                } else if (i == -2) {
-                       DEBUG(0,("%s: command abbreviation ambiguous\n",tok));
+                       d_printf("%s: command abbreviation ambiguous\n",tok);
                } else {
-                       DEBUG(0,("%s: command not found\n",tok));
+                       d_printf("%s: command not found\n",tok);
                }
        }
+       
+       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++;
        }
+}
+
+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);
 
-       if (count == 2) {
-               free(matches[0]);
-               matches[0] = strdup(matches[1]);
+       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;
+               pstr_sprintf(dirmask, "%s%*s*", cur_dir, i-1, text);
+       } else
+               pstr_sprintf(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
+ Make sure we swallow keepalives during idle time.
 ****************************************************************************/
+
 static void readline_callback(void)
 {
        fd_set fds;
@@ -1799,17 +2425,22 @@ static void readline_callback(void)
 
        t = time(NULL);
 
-       if (t - last_t < 5) return;
+       if (t - last_t < 5)
+               return;
 
        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
@@ -1823,25 +2454,28 @@ static void readline_callback(void)
        cli_chkpath(cli, "\\");
 }
 
-
 /****************************************************************************
-process commands on stdin
+ Process commands on stdin.
 ****************************************************************************/
+
 static void process_stdin(void)
 {
-       char *ptr;
+       const char *ptr;
 
        while (1) {
                fstring tok;
-               fstring prompt;
-               char *line;
+               fstring the_prompt;
+               char *cline;
+               pstring line;
                int i;
                
                /* display a prompt */
-               slprintf(prompt, sizeof(prompt), "smb: %s> ", cur_dir);
-               line = smb_readline(prompt, readline_callback, completion_fn);
-
-               if (!line) break;
+               slprintf(the_prompt, sizeof(the_prompt)-1, "smb: %s> ", cur_dir);
+               cline = smb_readline(the_prompt, readline_callback, completion_fn);
+                       
+               if (!cline) break;
+               
+               pstrcpy(line, cline);
 
                /* special case - first char is ! */
                if (*line == '!') {
@@ -1851,38 +2485,37 @@ static void process_stdin(void)
       
                /* and get the first part of the command */
                ptr = line;
-               if (!next_token(&ptr,tok,NULL,sizeof(tok))) continue;
+               if (!next_token_nr(&ptr,tok,NULL,sizeof(tok))) continue;
 
                if ((i = process_tok(tok)) >= 0) {
                        commands[i].fn();
                } else if (i == -2) {
-                       DEBUG(0,("%s: command abbreviation ambiguous\n",tok));
+                       d_printf("%s: command abbreviation ambiguous\n",tok);
                } else {
-                       DEBUG(0,("%s: command not found\n",tok));
+                       d_printf("%s: command not found\n",tok);
                }
        }
 }
 
-
 /***************************************************** 
-return a connection to a server
+ Return a connection to a server.
 *******************************************************/
-struct cli_state *do_connect(char *server, char *share)
+
+static struct cli_state *do_connect(const char *server, const char *share)
 {
        struct cli_state *c;
        struct nmb_name called, calling;
-       char *server_n;
+       const char *server_n;
        struct in_addr ip;
-       extern struct in_addr ipzero;
        fstring servicename;
        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;
-               sharename = strchr(server,'\\');
+               sharename = strchr_m(server,'\\');
                if (!sharename) return NULL;
                *sharename = 0;
                sharename++;
@@ -1890,31 +2523,33 @@ struct cli_state *do_connect(char *server, char *share)
 
        server_n = server;
        
-       ip = ipzero;
+       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:
-       ip = ipzero;
+       zero_ip(&ip);
        if (have_ip) ip = dest_ip;
 
        /* have to open a new connection */
-       if (!(c=cli_initialise(NULL)) || (cli_set_port(c, port) == 0) ||
+       if (!(c=cli_initialise(NULL)) || (cli_set_port(c, port) != port) ||
            !cli_connect(c, server_n, &ip)) {
-               DEBUG(0,("Connection to %s failed\n", server_n));
+               d_printf("Connection to %s failed\n", server_n);
                return NULL;
        }
 
        c->protocol = max_protocol;
+       c->use_kerberos = use_kerberos;
+       cli_setup_signing_state(c, cmdline_auth_info.signing_state);
+               
 
        if (!cli_session_request(c, &calling, &called)) {
                char *p;
-               DEBUG(0,("session request to %s failed (%s)\n", 
-                        called.name, cli_errstr(c)));
+               d_printf("session request to %s failed (%s)\n", 
+                        called.name, cli_errstr(c));
                cli_shutdown(c);
-               free(c);
-               if ((p=strchr(called.name, '.'))) {
+               if ((p=strchr_m(called.name, '.'))) {
                        *p = 0;
                        goto again;
                }
@@ -1928,9 +2563,8 @@ struct cli_state *do_connect(char *server, char *share)
        DEBUG(4,(" session request ok\n"));
 
        if (!cli_negprot(c)) {
-               DEBUG(0,("protocol negotiation failed\n"));
+               d_printf("protocol negotiation failed\n");
                cli_shutdown(c);
-               free(c);
                return NULL;
        }
 
@@ -1938,43 +2572,41 @@ struct cli_state *do_connect(char *server, char *share)
                char *pass = getpass("Password: ");
                if (pass) {
                        pstrcpy(password, pass);
+                       got_pass = 1;
                }
        }
 
        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] || 
-                   !cli_session_setup(c, "", "", 0, "", 0, workgroup)) { 
-                       DEBUG(0,("session setup failed: %s\n", cli_errstr(c)));
+               if (password[0] || !username[0] || use_kerberos ||
+                   !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);
-                       free(c);
                        return NULL;
                }
-               DEBUG(0,("Anonymous login successful\n"));
+               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"));
 
        if (!cli_send_tconX(c, sharename, "?????",
                            password, strlen(password)+1)) {
-               DEBUG(0,("tree connect failed: %s\n", cli_errstr(c)));
+               d_printf("tree connect failed: %s\n", cli_errstr(c));
                cli_shutdown(c);
-               free(c);
                return NULL;
        }
 
@@ -1983,136 +2615,59 @@ struct cli_state *do_connect(char *server, char *share)
        return c;
 }
 
-
 /****************************************************************************
-  process commands from the client
+ Process commands from the client.
 ****************************************************************************/
-static BOOL process(char *base_directory)
+
+static int process(char *base_directory)
 {
+       int rc = 0;
+
        cli = do_connect(desthost, service);
        if (!cli) {
-               return(False);
+               return 1;
        }
 
        if (*base_directory) do_cd(base_directory);
        
        if (cmdstr) {
-               process_command_string(cmdstr);
+               rc = process_command_string(cmdstr);
        } else {
                process_stdin();
        }
   
        cli_shutdown(cli);
-       return(True);
+       return rc;
 }
 
 /****************************************************************************
-usage on the program
+ Handle a -L query.
 ****************************************************************************/
-static void usage(char *pname)
+
+static int do_host_query(char *query_host)
 {
-  DEBUG(0,("Usage: %s service <password> [options]", pname));
+       cli = do_connect(query_host, "IPC$");
+       if (!cli)
+               return 1;
 
-  DEBUG(0,("\nVersion %s\n",VERSION));
-  DEBUG(0,("\t-s smb.conf           pathname to smb.conf file\n"));
-  DEBUG(0,("\t-O socket_options     socket options to use\n"));
-  DEBUG(0,("\t-R name resolve order use these name resolution services only\n"));
-  DEBUG(0,("\t-M host               send a winpopup message to the host\n"));
-  DEBUG(0,("\t-i scope              use this NetBIOS scope\n"));
-  DEBUG(0,("\t-N                    don't ask for a password\n"));
-  DEBUG(0,("\t-n netbios name.      Use this name as my netbios name\n"));
-  DEBUG(0,("\t-d debuglevel         set the debuglevel\n"));
-  DEBUG(0,("\t-P                    connect to service as a printer\n"));
-  DEBUG(0,("\t-p port               connect to the specified port\n"));
-  DEBUG(0,("\t-l log basename.      Basename for log/debug files\n"));
-  DEBUG(0,("\t-h                    Print this help message.\n"));
-  DEBUG(0,("\t-I dest IP            use this IP to connect to\n"));
-  DEBUG(0,("\t-E                    write messages to stderr instead of stdout\n"));
-  DEBUG(0,("\t-U username           set the network username\n"));
-  DEBUG(0,("\t-L host               get a list of shares available on a host\n"));
-  DEBUG(0,("\t-t terminal code      terminal i/o code {sjis|euc|jis7|jis8|junet|hex}\n"));
-  DEBUG(0,("\t-m max protocol       set the max protocol level\n"));
-  DEBUG(0,("\t-A filename           get the credentials from a file\n"));
-  DEBUG(0,("\t-W workgroup          set the workgroup name\n"));
-  DEBUG(0,("\t-T<c|x>IXFqgbNan      command line tar\n"));
-  DEBUG(0,("\t-D directory          start from directory\n"));
-  DEBUG(0,("\t-c command string     execute semicolon separated commands\n"));
-  DEBUG(0,("\t-b xmit/send buffer   changes the transmit/send buffer (default: 65520)\n"));
-  DEBUG(0,("\n"));
-}
+       browse_host(True);
 
+       if (port != 139) {
 
-/****************************************************************************
-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;
-       }
+               /* Workgroups simply don't make sense over anything
+                  else but port 139... */
 
-       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);
-               }
+               cli_shutdown(cli);
+               port = 139;
+               cli = do_connect(query_host, "IPC$");
        }
-       pstrcpy(password, pass);
-       if (close_it)
-               close(fd);
-}      
-
-
 
-/****************************************************************************
-handle a -L query
-****************************************************************************/
-static int do_host_query(char *query_host)
-{
-       cli = do_connect(query_host, "IPC$");
-       if (!cli)
+       if (cli == NULL) {
+               d_printf("NetBIOS over TCP disabled -- no workgroup available\n");
                return 1;
+       }
 
-       browse_host(True);
-       list_servers(workgroup);
+       list_servers(lp_workgroup());
 
        cli_shutdown(cli);
        
@@ -2121,14 +2676,19 @@ static int do_host_query(char *query_host)
 
 
 /****************************************************************************
-handle a tar operation
+ 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;
 
@@ -2142,28 +2702,34 @@ static int do_tar_op(char *base_directory)
 }
 
 /****************************************************************************
-handle a message operation
+ Handle a message operation.
 ****************************************************************************/
+
 static int do_message_op(void)
 {
        struct in_addr ip;
        struct nmb_name called, calling;
+       fstring server_name;
+       char name_type_hex[10];
 
-       ip = ipzero;
-
-       make_nmb_name(&calling, global_myname, 0x0);
+       make_nmb_name(&calling, global_myname(), 0x0);
        make_nmb_name(&called , desthost, name_type);
 
-       ip = ipzero;
+       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) == 0) || !cli_connect(cli, desthost, &ip)) {
-               DEBUG(0,("Connection to %s failed\n", desthost));
+       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;
        }
 
        if (!cli_session_request(cli, &calling, &called)) {
-               DEBUG(0,("session request failed\n"));
+               d_printf("session request failed\n");
                cli_shutdown(cli);
                return 1;
        }
@@ -2175,24 +2741,66 @@ 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[])
 {
+       extern BOOL AllowDebugChange;
        fstring base_directory;
-       char *pname = argv[0];
        int opt;
-       extern FILE *dbf;
-       extern char *optarg;
-       extern int optind;
        pstring query_host;
        BOOL message = False;
        extern char tar_type;
-       static pstring servicesf = CONFIGFILE;
        pstring term_code;
-       pstring new_name_resolve_order;
+       static const char *new_name_resolve_order = NULL;
+       poptContext pc;
        char *p;
+       int rc = 0;
+       fstring new_workgroup;
+       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);
@@ -2202,300 +2810,148 @@ 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 = stderr;
-               else if(strncmp(argv[opt], "-s", 2) == 0) {
-                       if(argv[opt][2] != '\0')
-                               pstrcpy(servicesf, &argv[opt][2]);
-                       else if(argv[opt+1] != NULL) {
-                               /*
-                                * At least one more arg left.
-                                */
-                               pstrcpy(servicesf, argv[opt+1]);
-                       } else {
-                               usage(pname);
-                               exit(1);
-                       }
-               }
-       }
-
-       TimeInit();
-       charset_initialise();
-
-       in_client = True;   /* Make sure that we tell lp_load we are */
-
-       if (!lp_load(servicesf,True,False,False)) {
-               fprintf(stderr, "Can't load %s - run testparm to debug it\n", servicesf);
-       }
        
-       codepage_initialise(lp_client_code_page());
-
-#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(username,'%'))) {
-                       *p = 0;
-                       pstrcpy(password,p+1);
-                       got_pass = True;
-                       memset(strchr(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);
-       }
+       /* initialize the workgroup name so we can determine whether or 
+          not it was set by a command line option */
+          
+       set_global_myworkgroup( "" );
 
-       if (*username == 0) {
-               pstrcpy(username,"GUEST");
-       }
+        /* set default debug level to 0 regardless of what smb.conf sets */
+       DEBUGLEVEL_CLASS[DBGC_ALL] = 0;
+       dbf = x_stderr;
+       x_setbuf( x_stderr, NULL );
 
-       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);
-                       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:")) != EOF) {
+       while ((opt = poptGetNextOpt(pc)) != -1) {
                switch (opt) {
-               case 's':
-                       pstrcpy(servicesf, 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);
-                               if (zero_ip(dest_ip))
+                               dest_ip = *interpret_addr2(poptGetOptArg(pc));
+                               if (is_zero_ip(dest_ip))
                                        exit(1);
                                have_ip = True;
                        }
                        break;
                case 'E':
-                       dbf = stderr;
-                       break;
-               case 'U':
-                       {
-                               char *lp;
-                               pstrcpy(username,optarg);
-                               if ((lp=strchr(username,'%'))) {
-                                       *lp = 0;
-                                       pstrcpy(password,lp+1);
-                                       got_pass = True;
-                                       memset(strchr(optarg,'%')+1,'X',strlen(password));
-                               }
-                       }
-                       break;
-
-               case 'A':
-                       {
-                               FILE *auth;
-                               fstring buf;
-                               uint16 len = 0;
-                               char *ptr, *val, *param;
-                               
-                               if ((auth=sys_fopen(optarg, "r")) == NULL)
-                               {
-                                       /* fail if we can't open the credentials file */
-                                       DEBUG(0,("ERROR: Unable to open credentials file!\n"));
-                                       exit (-1);
-                               }
-                                
-                               while (!feof(auth))
-                               {  
-                                       /* get a line from the file */
-                                       if (!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 (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));
-                               }
-                               fclose(auth);
-                       }
+                       dbf = x_stderr;
+                       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);
-                               exit(1);
+                       /* We must use old option processing for this. Find the
+                        * position of the -T option in the raw argv[]. */
+                       {
+                               int i, optnum;
+                               for (i = 1; i < argc; i++) {
+                                       if (strncmp("-T", argv[i],2)==0)
+                                               break;
+                               }
+                               i++;
+                               if (!(optnum = tar_parseargs(argc, argv, poptGetOptArg(pc), i))) {
+                                       poptPrintUsage(pc, stderr, 0);
+                                       exit(1);
+                               }
+                               /* Now we must eat (optnum - i) options - they have
+                                * been processed by tar_parseargs().
+                                */
+                               optnum -= i;
+                               for (i = 0; i < optnum; i++)
+                                       poptGetOptArg(pc);
                        }
                        break;
                case 'D':
-                       pstrcpy(base_directory,optarg);
-                       break;
-               case 'c':
-                       cmdstr = optarg;
-                       got_pass = True;
-                       break;
-               case 'b':
-                       io_bufsize = MAX(1, atoi(optarg));
+                       fstrcpy(base_directory,poptGetOptArg(pc));
                        break;
-               default:
-                       usage(pname);
+               }
+       }
+
+       poptGetArg(pc);
+       
+       /*
+        * Don't load debug level from smb.conf. It should be
+        * set by cmdline arg or remain default (0)
+        */
+       AllowDebugChange = False;
+       
+       /* save the workgroup...
+       
+          FIXME!! do we need to do tyhis for other options as well 
+          (or maybe a generic way to keep lp_load() from overwriting 
+          everything)?  */
+       
+       fstrcpy( new_workgroup, lp_workgroup() );               
+       
+       if (!lp_load(dyn_CONFIGFILE,True,False,False)) {
+               fprintf(stderr, "%s: Can't load %s - run testparm to debug it\n",
+                       argv[0], dyn_CONFIGFILE);
+       }
+       
+       load_interfaces();
+       
+       if ( strlen(new_workgroup) != 0 )
+               set_global_myworkgroup( new_workgroup );
+
+       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) { 
+               cmdline_auth_info.got_pass = True;
+               pstrcpy(cmdline_auth_info.password,poptGetArg(pc));  
+       }
 
-       if(*new_name_resolve_order)
-               lp_set_name_resolve_order(new_name_resolve_order);
+       init_names();
 
-       if (*term_code)
-               interpret_coding_system(term_code);
+       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);
        }
 
-       DEBUG( 3, ( "Client started (version %s).\n", VERSION ) );
+       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", SAMBA_VERSION_STRING));
 
        if (tar_type) {
                if (cmdstr)
@@ -2503,7 +2959,7 @@ static int do_message_op(void)
                return do_tar_op(base_directory);
        }
 
-       if ((p=strchr(query_host,'#'))) {
+       if ((p=strchr_m(query_host,'#'))) {
                *p = 0;
                p++;
                sscanf(p, "%x", &name_type);
@@ -2517,9 +2973,9 @@ static int do_message_op(void)
                return do_message_op();
        }
        
-       if (!process(base_directory)) {
-               return(1);
+       if (process(base_directory)) {
+               return 1;
        }
 
-       return(0);
+       return rc;
 }