Avoiding to write uninitialized bytes to the lock file.

Patch by Kamil Dudka.

git-svn-id: svn://svn.savannah.gnu.org/nano/trunk/nano@5108 35c25a1d-7b9e-4130-9fde-d3aeb78583b8

diff --git a/ChangeLog b/ChangeLog
index ac4c67784579dac704961d25f6247aca67faa5d9..74e735bbfa0103b8f8a0fb5f442a06e52250c946 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,7 @@
+2015-02-01  Kamil Dudka  <kdudka@redhat.com>
+       * src/files.c (write_lockfile): Avoid writing uninitialized bytes to
+       the lock file -- a simple null_at() would not initialize the buffer.
+
 2015-02-01  Benno Schulenberg  <bensberg@justemail.net>
        * src/winio.c (do_credits): Add a general entry for all translators.
        * src/nano.c (version), src/winio.c (do_credits): Update the copyright

diff --git a/src/files.c b/src/files.c
index 3ac53d81fb6a9c400572e46e3a24d9740ac06ad9..90c6b0cc5f198911400816871d5408ccf8e7fea1 100644 (file)
--- a/src/files.c
+++ b/src/files.c
@@ -195,7 +195,7 @@ int write_lockfile(const char *lockfilename, const char *origfilename, bool modi
      * our lockfile' message in here...
      *
      * This is likely very wrong, so this is a WIP. */
-    null_at(&lockdata, lockdatalen);
+    memset(lockdata, 0, lockdatalen);
     lockdata[0] = 0x62;
     lockdata[1] = 0x30;
     lockdata[24] = mypid % 256;