[Bug 377] Some PDF files will cause sendfax to segfault

Now they won't. Thanks Patrice!

diff --git a/util/SendFaxClient.c++ b/util/SendFaxClient.c++
index 6946c6d8f0b456958291c9c75e41050b4f480a0a..fa6c69b4d23299a73a485624849f99ecb2a3faf4 100644 (file)
--- a/util/SendFaxClient.c++
+++ b/util/SendFaxClient.c++
@@ -783,7 +783,7 @@ SendFaxClient::estimatePostScriptPages(const char* filename)
                rewind(fd);
                char *cp = line;
                int len;
-               while ((len = fread(cp, 1, endbuf-cp, fd)) > 0) {
+               while ((len = fread(cp, 1, endbuf-cp, fd)) > slen+line-cp) {
                     endbuf = cp+len;    // Will only change on the last pass.
                    cp = line;
                    while ((cp = (char *) memchr(cp, '/', endbuf-cp-slen))) {