Minor style fix: Test for null bidder object before trying to

allocate state.

SVN-Revision: 398

diff --git a/libarchive/archive_read_support_compression_program.c b/libarchive/archive_read_support_compression_program.c
index 884b5493072be1af9f1bef93d45923c8e8e86a1a..6a9823a389c6ebe76d7de0a2a8a4a4c7e8b3fe6b 100644 (file)
--- a/libarchive/archive_read_support_compression_program.c
+++ b/libarchive/archive_read_support_compression_program.c
@@ -109,12 +109,12 @@ archive_read_support_compression_program(struct archive *_a, const char *cmd)
        struct archive_read_filter_bidder *bidder = __archive_read_get_bidder(a);
        struct program_bidder *state;
 
-       state = (struct program_bidder *)calloc(sizeof (*state), 1);
+       if (bidder == NULL)
+               return (ARCHIVE_FATAL);
 
+       state = (struct program_bidder *)calloc(sizeof (*state), 1);
        if (state == NULL)
                return (ARCHIVE_FATAL);
-       if (bidder == NULL)
-               return (ARCHIVE_FATAL);
 
        state->cmd = strdup(cmd);
        state->bid = INT_MAX;