]> git.ipfire.org Git - thirdparty/dovecot/core.git/commitdiff
lib-smtp: address: Remove useless local variable initialization in smtp_address_clone().
authorStephan Bosch <stephan.bosch@dovecot.fi>
Sat, 20 Jan 2018 15:48:03 +0000 (16:48 +0100)
committerVille Savolainen <ville.savolainen@dovecot.fi>
Tue, 13 Mar 2018 09:07:03 +0000 (11:07 +0200)
Fixes complaint from scan-build.

src/lib-smtp/smtp-address.c

index a277e22b7b1bcc85e261981d841bf2a8121be84a..9969c36a974d3c69337d75e300fb609c24837514 100644 (file)
@@ -603,15 +603,14 @@ struct smtp_address *
 smtp_address_clone(pool_t pool, const struct smtp_address *src)
 {
        struct smtp_address *new;
-       size_t size, lpsize, dsize;
-       char *data, *localpart, *domain;
+       size_t size, lpsize, dsize = 0;
+       char *data, *localpart, *domain = NULL;
 
        if (smtp_address_isnull(src))
                return NULL;
 
        /* @UNSAFE */
 
-       lpsize = dsize = 0;
        size = sizeof(struct smtp_address);
        lpsize = strlen(src->localpart) + 1;
        size = MALLOC_ADD(size, lpsize);
@@ -620,13 +619,10 @@ smtp_address_clone(pool_t pool, const struct smtp_address *src)
                size = MALLOC_ADD(size, dsize);
        }
 
-       localpart = domain = NULL;
        data = p_malloc(pool, size);
        new = (struct smtp_address *)data;
-       if (lpsize > 0) {
-               localpart = PTR_OFFSET(data, sizeof(*new));
-               memcpy(localpart, src->localpart, lpsize);
-       }
+       localpart = PTR_OFFSET(data, sizeof(*new));
+       memcpy(localpart, src->localpart, lpsize);
        if (dsize > 0) {
                domain = PTR_OFFSET(data, sizeof(*new) + lpsize);
                memcpy(domain, src->domain, dsize);