Offset into the file can be size_t and add some casts to remove warnings.
authornicm <nicm@openbsd.org>
Mon, 5 Oct 2015 19:50:38 +0000 (19:50 +0000)
committernicm <nicm@openbsd.org>
Mon, 5 Oct 2015 19:50:38 +0000 (19:50 +0000)
usr.bin/file/magic-test.c
usr.bin/file/magic.h

index f2c72ea..9f239c1 100644 (file)
@@ -1,4 +1,4 @@
-/* $OpenBSD: magic-test.c,v 1.13 2015/08/12 09:39:43 nicm Exp $ */
+/* $OpenBSD: magic-test.c,v 1.14 2015/10/05 19:50:38 nicm Exp $ */
 
 /*
  * Copyright (c) 2015 Nicholas Marriott <nicm@openbsd.org>
@@ -1135,7 +1135,7 @@ magic_test_line(struct magic_line *ml, struct magic_state *ms)
        else {
                wanted = ml->indirect_offset;
                if (ml->indirect_relative) {
-                       if (wanted < 0 && -wanted > ms->offset)
+                       if (wanted < 0 && (size_t)-wanted > ms->offset)
                                return (0);
                        if (wanted > 0 && ms->offset + wanted > ms->size)
                                return (0);
@@ -1186,14 +1186,14 @@ magic_test_line(struct magic_line *ml, struct magic_state *ms)
        }
 
        if (ml->offset_relative) {
-               if (wanted < 0 && -wanted > ms->offset)
+               if (wanted < 0 && (size_t)-wanted > ms->offset)
                        return (0);
                if (wanted > 0 && ms->offset + wanted > ms->size)
                        return (0);
                offset = ms->offset + wanted;
        } else
                offset = wanted;
-       if (offset < 0 || offset > ms->size)
+       if (offset < 0 || (size_t)offset > ms->size)
                return (0);
        ms->offset = offset;
 
@@ -1213,7 +1213,7 @@ magic_test_line(struct magic_line *ml, struct magic_state *ms)
        if (ml->mimetype != NULL)
                ms->mimetype = ml->mimetype;
 
-       magic_warn(ml, "test %s/%c matched at offset %llu: '%s'",
+       magic_warn(ml, "test %s/%c matched at offset %zu: '%s'",
            ml->type_string, ml->test_operator, ms->offset,
            ml->result == NULL ? "" : ml->result);
 
index 7fa2f37..106e90a 100644 (file)
@@ -1,4 +1,4 @@
-/* $OpenBSD: magic.h,v 1.9 2015/10/02 18:06:27 deraadt Exp $ */
+/* $OpenBSD: magic.h,v 1.10 2015/10/05 19:50:38 nicm Exp $ */
 
 /*
  * Copyright (c) 2015 Nicholas Marriott <nicm@openbsd.org>
@@ -160,7 +160,7 @@ struct magic_state {
 
        const char              *base;
        size_t                   size;
-       int64_t                  offset;
+       size_t                   offset;
 };
 
 #define MAGIC_TEST_TEXT 0x1