diff options
| author | Andrew Bartlett <abartlet@samba.org> | 2002-07-20 13:02:47 +0000 |
|---|---|---|
| committer | Andrew Bartlett <abartlet@samba.org> | 2002-07-20 13:02:47 +0000 |
| commit | 29075c97d3b7111e2565ede1cd0f000fd2534375 (patch) | |
| tree | 683ef138ee8a4ddd5cf078222ca433fc808d6423 /source3/client | |
| parent | 9175bd2fe79684066a08e09860cf3a0bdb02d5c5 (diff) | |
| download | samba-29075c97d3b7111e2565ede1cd0f000fd2534375.tar.gz samba-29075c97d3b7111e2565ede1cd0f000fd2534375.tar.bz2 samba-29075c97d3b7111e2565ede1cd0f000fd2534375.zip | |
More fixes towards warnings on the IRIX compiler
(and yes, some of these are real bugs)
In particular, the samr code was doing an &foo of various types, to a function
that assumed uint32. If time_t isn't 32 bits long, that broke.
They are assignment compatible however, so use that and an intermediate
variable.
Andrew Bartlett
(This used to be commit 30d0998c8c1a1d4de38ef0fbc83c2b763e05a3e6)
Diffstat (limited to 'source3/client')
| -rw-r--r-- | source3/client/smbspool.c | 13 |
1 files changed, 5 insertions, 8 deletions
diff --git a/source3/client/smbspool.c b/source3/client/smbspool.c index b78d9d22a8..7804669cc8 100644 --- a/source3/client/smbspool.c +++ b/source3/client/smbspool.c @@ -284,18 +284,15 @@ smb_connect(char *workgroup, /* I - Workgroup */ nt_status = cli_full_connection(&c, myname, server, NULL, 0, share, "?????", username, lp_workgroup(), password, 0); - if (NT_STATUS_IS_OK(nt_status)) { - return c; - } else { + if (!NT_STATUS_IS_OK(nt_status)) { fprintf(stderr, "ERROR: Connection failed with error %s\n", nt_errstr(nt_status)); return NULL; } - - /* - * Return the new connection... - */ - + /* + * Return the new connection... + */ + return (c); } |