[pdf-devel] Patch for FS#94 missing warnings in filter utility

[gerel]

Hi hackers,

I attach a patch for the task in question.

##
# Bazaar merge directive format 2 (Bazaar 0.90)
# revision_id: address@hidden
# target_branch: file:///home/gerel/PROJECTS/libgnupdf/trunk/
# testament_sha1: 34dee3b07cc9994647497f38bdcef8c12eb90415
# timestamp: 2009-01-27 11:14:47 -0300
# base_revision_id: address@hidden
# 
# Begin patch
=== modified file 'ChangeLog'
--- ChangeLog   2009-01-26 22:13:35 +0000
+++ ChangeLog   2009-01-27 14:14:32 +0000
@@ -1,3 +1,7 @@
+2009-01-27  Gerardo E. Gidoni  <address@hidden>
+
+       * utils/pdf-filter.c: added missing status code checks.
+
 2009-01-26  Jose E. Marchesi  <address@hidden>
 
        * doc/gnupdf.texi (Implementation Limits): New chapter.

=== modified file 'utils/pdf-filter.c'
--- utils/pdf-filter.c  2009-01-24 20:38:14 +0000
+++ utils/pdf-filter.c  2009-01-27 14:13:44 +0000
@@ -1,4 +1,4 @@
-/* -*- mode: C -*- Time-stamp: "2009-01-24 17:33:37 gerel"
+/* -*- mode: C -*- Time-stamp: "2009-01-27 11:09:42 gerel"
  *
  *       File:         pdf-filter.c
  *       Date:         Tue Jul 10 18:42:07 2007
@@ -195,14 +195,20 @@
       do
         {
           ret = pdf_stm_read (stm, buf, BUF_SIZE, &read_bytes);
+          if (ret != PDF_OK)
+            {
+              pdf_error (ret, stderr, "reading from stream");
+              exit (1);
+            }            
+
           if (write_pdf_fsys)
             {
               ret = pdf_stm_write (fsys_stm, buf, read_bytes, &written_bytes);
               if (ret != PDF_OK)
                 {
-                  pdf_error (ret, stderr, "while writing to stream");
+                  pdf_error (ret, stderr, "writing to stream");
                   exit (1);
-                }            
+                }
             }
           else
             {
@@ -223,12 +229,24 @@
           if (read_pdf_fsys)
             {
               ret = pdf_stm_read (fsys_stm, buf, BUF_SIZE, &read_bytes);
+              if (ret != PDF_OK)
+                {
+                  pdf_error (ret, stderr, "reading from stream");
+                  exit (1);
+                }            
+
             }
           else
             {
               read_bytes = fread (buf, 1, BUF_SIZE, stdin);
             }
           ret = pdf_stm_write (stm, buf, read_bytes, &written_bytes);
+          if (ret != PDF_OK)
+            {
+              pdf_error (ret, stderr, "writing to stream");
+              exit (1);
+            }            
+
         }
       while (read_bytes == BUF_SIZE);
     }

# Begin bundle
IyBCYXphYXIgcmV2aXNpb24gYnVuZGxlIHY0CiMKQlpoOTFBWSZTWa8rgeYABCPfgAF1cXf//1+M
xAC////6YAbOJ806AANAKoPkBhKmiGqB6jTxNJ5TyeqNDT1MAgGNIeoZPUGVTRo0bUPKDBGhoAaN
AGgAADmjJiYAJiMCNMCDEYJkwCMEkQUZNT0TIaMQyMg0AAAA0AbVUyAAaA0aA0AA0AAAACpRATJo
AQTEp+IJlPBTam0QADNSBQPDrj6p+i9BC0XTasYqubg2rT2KHXxxOQgu0ZwvEHJVCdDC0hlVZeim
m5GwkTahkGoSVNMYE3TGOciTl6TiAJl1MkvCv5WXIHoQVfivgOPNiF4ajIGLDr3rAz2ozU09t4YH
eDa19xohRBa5KMJA7tSBTeKWRy+vIrE/X9fcgxVkp/FhFY8Icax8ShApQEGuBIuVEfFDCmNTpsHN
RDVUXxonVyMRGOTwaRkq2QOB3JvoLrtTrWpFylGakT+c9vTNRSn1+Xc5HsO1k83sJoenNU8uxWRN
jA7nY6nbO546SwJitd+CErwaBJZ7wqtnNq1ju/aFc3FVZQGkmfEpyUIdrnIl9i3xfsTBaWKJrKc3
vMXa62Ny4uimSrCkSe7ri32FxNpMWt/xo+HbepkmckLGQJJQmfuKQwFTBSUZrAZ8iAKHFBINSALU
Ec17Q0JVk3tcbkLkova7FDN89SvjubJXKMWLNve8luPNXTqWJtu0c5PLhWRNEuUZKryUoyNzu2F3
ClrGjAsxWX0cm9deyyl8wapU1M1ULFyw19q4XmbiwZa966qmjyotXaNlGSZrW6povYXS0mtUrYUr
RqWrFrqcWtY7zmrfuNhLM1tIyUQyZs++9g9CrUxS1J9zova3D5RY61XjNTOZGxfFjs2b1pg2nUVr
XZ0zzztM2FFkuNS1ez22XzC3WvTrvybVDkaSi3JymLXbHBsKtUwmuu3P7lN27Rib6K3NNWyeHofq
1Vb25+50ck1MHFO9seolmG9XniixDRhi34zFjV0b7OLdG2Te0LVqpR1OE/hOPZpxMdWtrN6+b9Tg
uW1cXFnuooUkTPGXtbUwcVzYz9yWVxXOLPgXLmLRo/Dxy4Y7M60NTSM8opFJLc43C/7SfA4dnqOS
kd1B9lI/rOLkpOGRRVSjJX8FeiilC2jPxfSPi2rExwjYHIETGGZJkghbZgZDM3LdmaE8fdOaJg/W
WH/J5FrZJTnQVm3bvulKTmsR3PFv9/m9bW+Pmxd/D03zceA8z5WxyjjLnBPnNv6SW6ErF/1P33ty
jo/Z2vc2vY+K9xE+p1u55yT6T4PBqXtFGa/NoUcMHY6pO33Nrwc7XJZKTfm2uFnRotOLYl8nN4rF
zLXIyXNH0yU5rO007lKSLHLLDNMffO+k+T0KM9V76LLPupuZH1O90dbLqeybo+Zuyo6KLHCUGT17
ZNFXXo8vqmDwU6k1Ti2pVE1mEleaUkbjPhV+uCJ+Fx8TwHeXxUiFkNFoCJJW1RsMm2ipOXiIYjLx
4yaMJ2GuMWZP0m+e2fnqU87IzKNbmtT1quRUrImcUR98HGAYLpZ9Cg4iKGHTDprf4JL8LiXRSKDs
R/pU1UeSe5OTb0J4fJ6XNc8pHqvP/MfCN8LVD71Oz3/AYYOa9a/v2L2K+Q8ie+Q8n5Tiqun09Ton
j3qUnfPzJ6j0ega2d62apZJzbVkke2Gr2LCx5qG5xL96hfJpF9ket1LC06ObFXCco7MLilJJdFyw
WKQqGLylTRkkBBfAsiAYRBK9WenRuW1+VBJtMqAowJXVFmEARGSWtAZ1JE8/RNqoaGiinE4nAsMu
vKVswjIa++5qKPS2l5x+fI8BaL1bhdNbjHlLkS4nrk1tzjNiklk9LRJMdXUsk+xr3ruI4GJMKF9U
4+nzp4ErevKKeg7aykZxYvkKz13SfBMC5vq4xqFIpZZLZ1VPU0WHr9qbBRaa2aVb+BwWmtoq1DJK
RT9KGHGVkwaWliSenUedE6l2W3w06J0ZpzkPxZKE/E2lHBkwn2J5HhMpG3c3Pyqqo95PRzFhC5rh
gFaP0/hmb/F3JFOFCQryuB5g

###

regards,
-gerel