]> sigrok.org Git - libsigrok.git/commitdiff
input/chronovu_la8: address file size data type nits
authorGerhard Sittig <redacted>
Sat, 12 May 2018 12:20:44 +0000 (14:20 +0200)
committerUwe Hermann <redacted>
Thu, 17 May 2018 20:25:40 +0000 (22:25 +0200)
Although the file format handled by this input module appears to be of
fixed size (8MiB plus 5 more bytes), it's more reliable to use a data
type for the file size that is larger than "an int". Although off_t
would be most portable, use uint64_t to match the code which passes the
parameter to the input module.

src/input/chronovu_la8.c

index 6607f3a7e11fba56452223edfebf565bfc7bbce0..e83d58229118c5503a16d07bd3afa4d91cb7fd13 100644 (file)
@@ -39,7 +39,7 @@ struct context {
 
 static int format_match(GHashTable *metadata, unsigned int *confidence)
 {
-       int size;
+       uint64_t size;
 
        /*
         * In the absence of a reliable condition like magic strings,
@@ -47,7 +47,7 @@ static int format_match(GHashTable *metadata, unsigned int *confidence)
         * rather weak a condition, signal "little confidence" and
         * optionally give precedence to better matches.
         */
-       size = GPOINTER_TO_INT(g_hash_table_lookup(metadata,
+       size = GPOINTER_TO_SIZE(g_hash_table_lookup(metadata,
                        GINT_TO_POINTER(SR_INPUT_META_FILESIZE)));
        if (size != CHRONOVU_LA8_FILESIZE)
                return SR_ERR;