[pdf-devel] Patch for memcpy() replacements

[gerel]

A patch to use memcpy() instead of strncpy() where it's not intended.

##
# Bazaar merge directive format 2 (Bazaar 0.90)
# revision_id: address@hidden
# target_branch: file:///home/gerel/PROJECTS/libgnupdf/trunk/
# testament_sha1: a13d71ac0bc97aa06d3e02ddd539a8d35e8fbc84
# timestamp: 2008-09-27 17:56:09 -0300
# base_revision_id: address@hidden
# 
# Begin patch
=== modified file 'ChangeLog'
--- ChangeLog   2008-09-26 22:52:29 +0000
+++ ChangeLog   2008-09-27 20:55:07 +0000
@@ -1,3 +1,9 @@
+2008-09-27  Gerardo E. Gidoni  <address@hidden>
+
+       * src/base/pdf-stm-f-null.c: using memcpy() instead of strncpy().
+
+       * src/base/pdf-stm.c: same.
+
 2008-09-27  Jose E. Marchesi  <address@hidden>
 
        * src/base/pdf-stm-f-ahex.c (pdf_stm_f_ahexdec_apply): Fix the

=== modified file 'src/base/pdf-stm-f-null.c'
--- src/base/pdf-stm-f-null.c   2008-09-26 18:56:50 +0000
+++ src/base/pdf-stm-f-null.c   2008-09-27 20:55:07 +0000
@@ -1,4 +1,4 @@
-/* -*- mode: C -*- Time-stamp: "08/09/26 20:52:10 jemarch"
+/* -*- mode: C -*- Time-stamp: "2008-09-27 17:52:19 gerel"
  *
  *       File:         pdf-stm-f-null.c
  *       Date:         Mon Jul  9 22:01:41 2007
@@ -64,9 +64,9 @@
 
   if (bytes_to_copy != 0)
     {
-      strncpy ((char *) out->data,
-               (char *) in->data,
-               bytes_to_copy);
+      memcpy ((char *) out->data,
+              (char *) in->data,
+              bytes_to_copy);
 
       in->rp = in->rp + bytes_to_copy;
       out->wp = out->wp + bytes_to_copy;

=== modified file 'src/base/pdf-stm.c'
--- src/base/pdf-stm.c  2008-09-22 21:41:52 +0000
+++ src/base/pdf-stm.c  2008-09-27 20:55:07 +0000
@@ -1,4 +1,4 @@
-/* -*- mode: C -*- Time-stamp: "08/09/22 23:29:02 jemarch"
+/* -*- mode: C -*- Time-stamp: "2008-09-27 17:52:37 gerel"
  *
  *       File:         pdf-stm.c
  *       Date:         Fri Jul  6 18:43:15 2007
@@ -161,9 +161,9 @@
           cache_size = stm->cache->wp - stm->cache->rp;
           to_copy_bytes = PDF_MIN(pending_bytes, cache_size);
 
-          strncpy ((char *) (buf + read_bytes),
-                   (char *) stm->cache->data + stm->cache->rp,
-                   to_copy_bytes);
+          memcpy ((char *) (buf + read_bytes),
+                  (char *) stm->cache->data + stm->cache->rp,
+                  to_copy_bytes);
           
           read_bytes += to_copy_bytes;
           stm->cache->rp += to_copy_bytes;
@@ -227,9 +227,9 @@
 
           if (to_write_bytes != 0)
             {
-              strncpy ((char *) tail_buffer->data + tail_buffer->wp,
-                       (char *) buf + written_bytes,
-                       to_write_bytes);
+              memcpy ((char *) tail_buffer->data + tail_buffer->wp,
+                      (char *) buf + written_bytes,
+                      to_write_bytes);
 
               written_bytes += to_write_bytes;
               tail_buffer->wp += to_write_bytes;

# Begin bundle
IyBCYXphYXIgcmV2aXNpb24gYnVuZGxlIHY0CiMKQlpoOTFBWSZTWbUoiVQAAxV/gBExEABRf///
WoQEAL////BQBZql6qACQaKwgCShPUmpoYamyR5oUaeoaB6gaAaAOaYmABMAEwAAmAAJgkUJop4V
P0KbZET1PKeTU09MkBmoGRoOaYmABMAEwAAmAAJgkkE00CaYRPQmmQGoZGppoaZNPU9SZINeQwjX
J6oDb56li8Xp6GYDYWzOutQBq104nURkmlAIpxibchJFoQJbHJJQQeE1xs/Vv5fgXjgPgbTqD5oN
L/F4vKSsB45SGd4tPrqVMaZzocaJxlpoIvt6CnLsXtwcU+TzmG9DGPxOGauB1uRXnqJWke9gGfYU
3cdV8WnLTl0OolZRxyQtwRYORC6uCA3WCpPlx3w4uWf2KLXlBlUPid5SLaDDgbP2dFYUBYrzn1Dj
eHgOANqIBoI/JHw+PR74H46FPIXZIgFYdMfGi7tsY0/MQazOHrLMvttXgMkQTmjI20Xb3dXkLJIT
hDAUayQniJCKYECLKTCQ3FUgHVx+AUMrLDuquQpQmJ1cqRVkZxuueSjmG4J3THWxYpGPn5u5ap4O
peQrC41lh+TILHxG5ZiytN+Q4SHgYiJlQqZ4U4PTzbqjSEagoCZmO3kRTRssLk8QxUMRCosFmc3m
ge4jC5M1TpXiqI1QvIFoxtIhsIEqi0yn7CKjqLj3KrkirVLE1BwG4ZF4REThM1HNGoeSxKwgQLCB
uxvpIKgbIoHwEQJSv1gPB5AzLigjgSEQKRpaxjcQLZ2OYqfeODMjIjaQCGM4kvYVZQakXBExLhPj
BUW460kzpkRErJRLgtNZeVvHoqRUZCdXTRINAYSKQmUD7ykYlEvHYGJOULiBcbhoPGpGHvRWqR4P
ULIgFZYNhE5xLTEmaB8RxeUWszTxHXXvQwMh94UCj4ryG5HewbmHNnqud7g3UQzDcezpUVIZw2d4
XcozgUvcKTk9mwTSL5MYNvjmKXFkmUF1krPEykYpgO62BQwPocEAOJI5BwOZmS8uBcd4jkbq1xKC
83iqPg/IHqGCkwu/3D0FGxBkD1eH26eA8wHHgVmJoVDHYdo83nQoK2KP25/8YgwdyR8xPOZzLC01
nYSOhiZHq4R2WC57McRUKEccTeE2M5Z5m07T5FKNhQj33ENpeBX2m0LTZwPEo5gtk4dF4ngbVBQG
pHvNFfdqMxzCLzSadayB2Fsy6XQfnTxUkKMvIwJl3h5EgCo+grhVwPVMlqMNcfaaS04BANw1aqEO
SxfxMOv6FwYHidR2jhjrO4zFeFh9V4r4K85OVZcHbmKJg6Q4cgLyDCY7pn38x6GVvsINYfl6RmnB
6VoG14ifNl0CNV3aJhsAL5VQaBtN6D3IGh5it1j/Xqg+AjoIvOL16dA5C2GtqjQY9gjQLtVYTRcK
ILckY6kRNdNYjECQsyYqipHp6YYH036aiwsNOiotWIUybxYXsJhOqYpNRUO6FYiVKG0uQgo1jkob
Jq4rs/2BpFfWI211IkVi8BBatkVmup53qKwwwR7otjboCoMwr40Ujf1eKTNtvNEtEQF+7nhzGF9W
w7opJ6C3sirmT7BViDhTPYQX8mhDIOUihhkyIjM15ZqCWiJhsOIhy1VIpKh43c8etgzExPaurTk8
pPwJwfKkvUNTrjgiIp4dg8oEegjH9f7AWpMykyLBw4Qd9/IB4Zys0uyEXiJLqEfcUxcNwTxr+y/g
Jpdax0cDmczn4I4CMtgEP/F3JFOFCQtSiJVA

###

cheers

-gerel