svn commit: samba r11256 - in branches/SAMBA_3_0/source: libsmb passdb

jra at samba.org jra at samba.org
Fri Oct 21 22:48:29 GMT 2005


Author: jra
Date: 2005-10-21 22:48:27 +0000 (Fri, 21 Oct 2005)
New Revision: 11256

WebSVN: http://websvn.samba.org/cgi-bin/viewcvs.cgi?view=rev&root=samba&rev=11256

Log:
Remove use of long long and strtoll in libsmbclient (we
can't assume long long is always there). Removed unused
var in new a/c rename code.
long long still used in eventlog code but Jerry has promised
to fix that.
Jeremy.

Modified:
   branches/SAMBA_3_0/source/libsmb/libsmbclient.c
   branches/SAMBA_3_0/source/passdb/pdb_smbpasswd.c


Changeset:
Modified: branches/SAMBA_3_0/source/libsmb/libsmbclient.c
===================================================================
--- branches/SAMBA_3_0/source/libsmb/libsmbclient.c	2005-10-21 22:48:15 UTC (rev 11255)
+++ branches/SAMBA_3_0/source/libsmb/libsmbclient.c	2005-10-21 22:48:27 UTC (rev 11256)
@@ -30,14 +30,13 @@
 /*
  * DOS Attribute values (used internally)
  */
-typedef struct DOS_ATTR_DESC
-{
-    int mode;
-    unsigned long long size;
-    time_t a_time;
-    time_t c_time;
-    time_t m_time;
-    unsigned long long inode;
+typedef struct DOS_ATTR_DESC {
+	int mode;
+	SMB_OFF_T size;
+	time_t a_time;
+	time_t c_time;
+	time_t m_time;
+	SMB_INO_T inode;
 } DOS_ATTR_DESC;
 
 
@@ -3738,36 +3737,36 @@
 		}
 
 		if (StrnCaseCmp(tok, "SIZE:", 5) == 0) {
-                        dad->size = strtoll(tok+5, NULL, 10);
+                        dad->size = (SMB_OFF_T)atof(tok+5);
 			continue;
 		}
 
 		if (StrnCaseCmp(tok, "A_TIME:", 7) == 0) {
-                        dad->a_time = strtoll(tok+7, NULL, 10);
+                        dad->a_time = (time_t)strtol(tok+7, NULL, 10);
 			continue;
 		}
 
 		if (StrnCaseCmp(tok, "C_TIME:", 7) == 0) {
-                        dad->c_time = strtoll(tok+7, NULL, 10);
+                        dad->c_time = (time_t)strtol(tok+7, NULL, 10);
 			continue;
 		}
 
 		if (StrnCaseCmp(tok, "M_TIME:", 7) == 0) {
-                        dad->m_time = strtoll(tok+7, NULL, 10);
+                        dad->m_time = (time_t)strtol(tok+7, NULL, 10);
 			continue;
 		}
 
 		if (StrnCaseCmp(tok, "INODE:", 6) == 0) {
-                        dad->inode = strtoll(tok+6, NULL, 10);
+                        dad->inode = (SMB_INO_T)atof(tok+6);
 			continue;
 		}
 	}
 }
 
-
 /***************************************************** 
-retrieve the acls for a file
+ Retrieve the acls for a file.
 *******************************************************/
+
 static int cacl_get(SMBCCTX *context, TALLOC_CTX *ctx, SMBCSRV *srv,
                     struct cli_state *ipc_cli, POLICY_HND *pol,
                     char *filename, char *attr_name, char *buf, int bufsize)
@@ -4201,8 +4200,8 @@
                                 if (determine_size) {
                                         p = talloc_asprintf(
                                                 ctx,
-                                                ",SIZE:%llu",
-                                                (unsigned long long) size);
+                                                ",SIZE:%.0f",
+                                                (double)size);
                                         if (!p) {
                                                 errno = ENOMEM;
                                                 return -1;
@@ -4210,15 +4209,15 @@
                                         n = strlen(p);
                                 } else {
                                         n = snprintf(buf, bufsize,
-                                                     ",SIZE:%llu",
-                                                     (unsigned long long) size);
+                                                     ",SIZE:%.0f",
+                                                     (double)size);
                                 }
                         } else if (StrCaseCmp(name, "size") == 0) {
                                 if (determine_size) {
                                         p = talloc_asprintf(
                                                 ctx,
-                                                "%llu",
-                                                (unsigned long long) size);
+                                                "%.0f",
+                                                (double)size);
                                         if (!p) {
                                                 errno = ENOMEM;
                                                 return -1;
@@ -4226,8 +4225,8 @@
                                         n = strlen(p);
                                 } else {
                                         n = snprintf(buf, bufsize,
-                                                     "%llu",
-                                                     (unsigned long long) size);
+                                                     "%.0f",
+                                                     (double)size);
                                 }
                         }
         
@@ -4356,8 +4355,8 @@
                                 if (determine_size) {
                                         p = talloc_asprintf(
                                                 ctx,
-                                                ",INODE:%llu",
-                                                (unsigned long long) ino);
+                                                ",INODE:%.0f",
+                                                (double)ino);
                                         if (!p) {
                                                 errno = ENOMEM;
                                                 return -1;
@@ -4365,15 +4364,15 @@
                                         n = strlen(p);
                                 } else {
                                         n = snprintf(buf, bufsize,
-                                                     ",INODE:%llu",
-                                                     (unsigned long long) ino);
+                                                     ",INODE:%.0f",
+                                                     (double) ino);
                                 }
                         } else if (StrCaseCmp(name, "inode") == 0) {
                                 if (determine_size) {
                                         p = talloc_asprintf(
                                                 ctx,
-                                                "%llu",
-                                                (unsigned long long) ino);
+                                                "%.0f",
+                                                (double) ino);
                                         if (!p) {
                                                 errno = ENOMEM;
                                                 return -1;
@@ -4381,8 +4380,8 @@
                                         n = strlen(p);
                                 } else {
                                         n = snprintf(buf, bufsize,
-                                                     "%llu",
-                                                     (unsigned long long) ino);
+                                                     "%.0f",
+                                                     (double) ino);
                                 }
                         }
         

Modified: branches/SAMBA_3_0/source/passdb/pdb_smbpasswd.c
===================================================================
--- branches/SAMBA_3_0/source/passdb/pdb_smbpasswd.c	2005-10-21 22:48:15 UTC (rev 11255)
+++ branches/SAMBA_3_0/source/passdb/pdb_smbpasswd.c	2005-10-21 22:48:27 UTC (rev 11256)
@@ -1481,7 +1481,6 @@
 					      SAM_ACCOUNT *old_acct,
 					      const char *newname)
 {
-	struct smbpasswd_privates *smbpasswd_state = (struct smbpasswd_privates*)my_methods->private_data;
 	pstring rename_script;
 	SAM_ACCOUNT *new_acct = NULL;
 	BOOL interim_account = False;



More information about the samba-cvs mailing list