r25184: Fix some C++ warnings and an uninitialized variable
authorVolker Lendecke <vlendec@samba.org>
Sat, 15 Sep 2007 20:24:35 +0000 (20:24 +0000)
committerGerald (Jerry) Carter <jerry@samba.org>
Wed, 10 Oct 2007 17:30:49 +0000 (12:30 -0500)
(This used to be commit b64df8a3c504ab7749c21ffb26e4771a9a0a328f)

source3/lib/charcnv.c
source3/lib/util_str.c
source3/smbd/msdfs.c
source3/smbd/nttrans.c
source3/smbd/trans2.c

index 26581ba305b576438c3ae81218b85b7024533d1b..a8fb95df06300539a24334cebbfa58d488d9c067 100644 (file)
@@ -1493,7 +1493,7 @@ static size_t pull_ucs2_base_talloc(TALLOC_CTX *ctx,
                        /* Have we got space to append the '\0' ? */
                        if (size <= dest_len) {
                                /* No, realloc. */
-                               dest = TALLOC_REALLOC(ctx, dest,
+                               dest = TALLOC_REALLOC_ARRAY(ctx, dest, char,
                                                dest_len+1);
                                if (!dest) {
                                        /* talloc fail. */
index c2eeb12544e2f12db06b502ca51754822479fc5f..db5775b79fd16790f39794f60134715e432ca8bb 100644 (file)
@@ -995,7 +995,7 @@ BOOL in_list(const char *s, const char *list, BOOL casesensitive)
        /* We know a token can't be larger
         * than the entire list. */
 
-       tok = SMB_MALLOC(bufsize+1);
+       tok = SMB_MALLOC_ARRAY(char, bufsize+1);
        if (!tok) {
                return False;
        }
index 56089fbe95d9f787fcef3737b471948c45b45664..69612dc571b474c1c812794152f150d1d9dec409 100644 (file)
@@ -1549,7 +1549,7 @@ struct junction_map *enum_msdfs_links(TALLOC_CTX *ctx, size_t *p_num_jn)
 {
        struct junction_map *jn = NULL;
        int i=0;
-       size_t jn_count;
+       size_t jn_count = 0;
        int sharecount = 0;
 
        *p_num_jn = 0;
index 14a11c3fe362d910e54fc3408a0a5ef89239ce23..dce0d6188d91f669c3e8752a24937d1b20911d4c 100644 (file)
@@ -634,7 +634,7 @@ void reply_ntcreate_and_X(connection_struct *conn,
                 */
 
                dir_name_len = strlen(dir_fsp->fsp_name);
-               fname = TALLOC_SIZE(ctx, dir_name_len+2);
+               fname = TALLOC_ARRAY(ctx, char, dir_name_len+2);
                if (!fname) {
                        reply_nterror(
                                req, NT_STATUS_NO_MEMORY);
@@ -1392,7 +1392,7 @@ static void call_nt_transact_create(connection_struct *conn,
                 */
 
                dir_name_len = strlen(dir_fsp->fsp_name);
-               fname = TALLOC_SIZE(ctx, dir_name_len+2);
+               fname = TALLOC_ARRAY(ctx, char, dir_name_len+2);
                if (!fname) {
                        reply_nterror(
                                req, NT_STATUS_NO_MEMORY);
index 478f60d094c6113557c9bbe092bf571648578693..ada4868bf887b35a6b66ac64bade89e78237cc75 100644 (file)
@@ -4209,7 +4209,7 @@ total_data=%u (should be %u)\n", (unsigned int)total_data, (unsigned int)IVAL(pd
 
                case SMB_QUERY_FILE_UNIX_LINK:
                        {
-                               char *buffer = TALLOC_SIZE(ctx, 1024);
+                               char *buffer = TALLOC_ARRAY(ctx, char, 1024);
 
                                if (!buffer) {
                                        reply_nterror(req, NT_STATUS_NO_MEMORY);