commit 1e939bafd88fba862db538176520e4c88bebe424
Author: Oswald Buddenhagen <o...@users.sf.net>
Date:   Sun Apr 2 12:03:16 2017 +0200

    don't use strncpy() to copy TUIDs
    
    latest since 77acc268, the code prior to these statements ensures that
    the full length is available, so just use memcpy(). the code for
    comparing TUIDs uses memcmp() anyway.

 src/drv_imap.c    | 2 +-
 src/drv_maildir.c | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/drv_imap.c b/src/drv_imap.c
index e89cea2..6d247b0 100644
--- a/src/drv_imap.c
+++ b/src/drv_imap.c
@@ -1060,7 +1060,7 @@ parse_fetch_rsp( imap_store_t *ctx, list_t *list, char *s 
ATTR_UNUSED )
                cur->gen.srec = 0;
                cur->gen.msgid = msgid;
                if (tuid)
-                       strncpy( cur->gen.tuid, tuid, TUIDL );
+                       memcpy( cur->gen.tuid, tuid, TUIDL );
                else
                        cur->gen.tuid[0] = 0;
                if (ctx->gen.uidnext <= uid) /* in case the server sends no 
UIDNEXT */
diff --git a/src/drv_maildir.c b/src/drv_maildir.c
index d174bdd..97a63c7 100644
--- a/src/drv_maildir.c
+++ b/src/drv_maildir.c
@@ -1132,7 +1132,7 @@ maildir_init_msg( maildir_store_t *ctx, maildir_message_t 
*msg, msg_t *entry )
        entry->msgid = 0; /* prevent deletion */
        msg->gen.size = entry->size;
        msg->gen.srec = 0;
-       strncpy( msg->gen.tuid, entry->tuid, TUIDL );
+       memcpy( msg->gen.tuid, entry->tuid, TUIDL );
        if (entry->recent)
                msg->gen.status |= M_RECENT;
        if (ctx->gen.opts & OPEN_FLAGS) {

------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________
isync-devel mailing list
isync-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/isync-devel

Reply via email to