- document Imager::Filter::Mandelbrot

[imager.git] / error.c

diff --git a/error.c b/error.c
index 1737bc421f4fa428f65043bbb1300b98a58ec27e..3dcab3cd1a9f5e0a1f544ecc1eb011881860e475 100644 (file)
--- a/error.c
+++ b/error.c
@@ -61,19 +61,19 @@ C).  The Perl level won't use all of this.
 =cut
 */
 
-#include "image.h"
+#include "imager.h"
 #include <stdio.h>
 #include <stdlib.h>
 
 /* we never actually use the last item - it's the NULL terminator */
 #define ERRSTK 20
-i_errmsg error_stack[ERRSTK];
-int error_sp = ERRSTK - 1;
+static i_errmsg error_stack[ERRSTK];
+static int error_sp = ERRSTK - 1;
 /* we track the amount of space used each string, so we don't reallocate 
    space unless we need to.
    This also means that a memory tracking library may see the memory 
    allocated for this as a leak. */
-int error_space[ERRSTK];
+static int error_space[ERRSTK];
 
 static i_error_cb error_cb;
 static i_failed_cb failed_cb;
@@ -95,7 +95,9 @@ void i_set_argv0(char const *name) {
   char *dupl;
   if (!name)
     return;
-  dupl = mymalloc(strlen(name)+1);
+  /* if the user has an existing string of MAXINT length then
+     the system is broken anyway */
+  dupl = mymalloc(strlen(name)+1); /* check 17jul05 tonyc */
   strcpy(dupl, name);
   if (argv0)
     myfree(argv0);
@@ -182,9 +184,14 @@ the mark.
 
 =item i_clear_error()
 
+=category Error handling
+
+Clears the error stack.
+
 Called by any imager function before doing any other processing.
 
-=cut */
+=cut
+*/
 void i_clear_error() {
 #ifdef IMAGER_DEBUG_MALLOC
   int i;
@@ -203,6 +210,8 @@ void i_clear_error() {
 /*
 =item i_push_error(int code, char const *msg)
 
+=category Error handling
+
 Called by an imager function to push an error message onto the stack.
 
 No message is pushed if the stack is full (since this means someone
@@ -222,9 +231,11 @@ void i_push_error(int code, char const *msg) {
   if (error_space[error_sp] < size) {
     if (error_stack[error_sp].msg)
       myfree(error_stack[error_sp].msg);
-    /* memory allocated on the following line is only ever release when 
+    /* memory allocated on the following line is only ever released when 
        we need a bigger string */
-    error_stack[error_sp].msg = mymalloc(size);
+    /* size is size (len+1) of an existing string, overflow would mean
+       the system is broken anyway */
+    error_stack[error_sp].msg = mymalloc(size); /* checked 17jul05 tonyc */
     error_space[error_sp] = size;
   }
   strcpy(error_stack[error_sp].msg, msg);
@@ -237,6 +248,8 @@ void i_push_error(int code, char const *msg) {
 /*
 =item i_push_errorvf(int code, char const *fmt, va_list ap)
 
+=category Error handling
+
 Intended for use by higher level functions, takes a varargs pointer
 and a format to produce the finally pushed error message.
 
@@ -259,6 +272,8 @@ void i_push_errorvf(int code, char const *fmt, va_list ap) {
 /*
 =item i_push_errorf(int code, char const *fmt, ...)
 
+=category Error handling
+
 A version of i_push_error() that does printf() like formating.
 
 =cut
@@ -270,6 +285,9 @@ void i_push_errorf(int code, char const *fmt, ...) {
   va_end(ap);
 }
 
+#ifdef IMAGER_I_FAILED
+#error "This isn't used and is untested"
+
 /*
 =item i_failed(char const *msg)
 
@@ -326,6 +344,8 @@ int i_failed(int code, char const *msg) {
   return 0;
 }
 
+#endif
+
 /*
 =back