X-Git-Url: http://mj.ucw.cz/gitweb/?a=blobdiff_plain;f=images%2Fobject.c;h=501b23d0575130e916ab12de9c8d4f60ce9508dd;hb=0db6e10eac28f38bfc3b325b13ad95107c58ce1e;hp=d817e1c92f10c519d087073e316c2f10fc5fd242;hpb=f8eb8bab9f7a35b8c3249517715134693e5920dc;p=libucw.git

diff --git a/images/object.c b/images/object.c
index d817e1c9..501b23d0 100644
--- a/images/object.c
+++ b/images/object.c
@@ -9,14 +9,14 @@
 #undef LOCAL_DEBUG
 
 #include "sherlock/sherlock.h"
-#include "lib/base224.h"
-#include "lib/mempool.h"
-#include "lib/fastbuf.h"
+#include <ucw/base224.h>
+#include <ucw/mempool.h>
+#include <ucw/fastbuf.h>
 #include "sherlock/object.h"
-#include "images/images.h"
-#include "images/object.h"
-#include "images/color.h"
-#include "images/signature.h"
+#include <images/images.h>
+#include <images/object.h>
+#include <images/color.h>
+#include <images/signature.h>
 #include <stdio.h>
 #include <string.h>
 
@@ -29,7 +29,7 @@ get_image_obj_info(struct image_obj_info *ioi, struct odes *o)
       DBG("Missing image info attribute");
       return 0;
     }
-  byte color_space[MAX_ATTR_SIZE], thumb_format[MAX_ATTR_SIZE];
+  byte color_space[16], thumb_format[16];
   UNUSED uns cnt = sscanf(v, "%d%d%s%d%d%d%s", &ioi->cols, &ioi->rows, color_space,
       &ioi->colors, &ioi->thumb_cols, &ioi->thumb_rows, thumb_format);
   ASSERT(cnt == 7);
@@ -48,9 +48,13 @@ get_image_obj_thumb(struct image_obj_info *ioi, struct odes *o, struct mempool *
       return 0;
     }
   uns count = 0;
+  uns max_len = 0;
   for (struct oattr *b = a; b; b = b->same)
-    count++;
-  byte buf[count * MAX_ATTR_SIZE], *b = buf;
+    {
+      count++;
+      max_len += strlen(b->val);
+    }
+  byte buf[max_len + 1], *b = buf;
   for (; a; a = a->same)
     b += base224_decode(b, a->val, strlen(a->val));
   ASSERT(b != buf);
@@ -81,16 +85,15 @@ read_image_obj_thumb(struct image_obj_info *ioi, struct fastbuf *fb, struct imag
   return img;
 error:
   DBG("Failed to decompress thumbnail: %s", io->thread->err_msg);
-  return NULL;  
+  return NULL;
 }
 
 void
 put_image_obj_signature(struct odes *o, struct image_signature *sig)
 {
-  /* signatures should be short enough to fit one attribute */
-  byte buf[MAX_ATTR_SIZE];
+  /* signatures should be short enough to in a single attribute */
   uns size = image_signature_size(sig->len);
-  ASSERT(MAX_ATTR_SIZE > BASE224_ENC_LENGTH(size));
+  byte buf[BASE224_ENC_LENGTH(size) + 1];
   buf[base224_encode(buf, (byte *)sig, size)] = 0;
   obj_set_attr(o, 'H', buf);
 }