Read the magic number into a byte array.

Apparently, in some C implementations, attempting to do an fread() into
a variable of a 32-bit unsigned integral type with a size of 1 and a
count of 4 returns 0 with an EOF indication; see GitHub pull request

We can make the size be the size of the variable and the count be 1, but
that means that the count returned by an fread() terminated by an EOF
will be 0, not the number of bytes successfully read, so the "truncated
dump file" message will give an invalid count:

	tcpdump: truncated dump file; tried to read 4 file header bytes,
	    only got 0

If, instead, we read into an array of 4 bytes, with a size of 1 and a
count of 4, we'll get the right short count back.

Pass the byte array to the file-type-specific "is this a file of this
type?" routines, so that if we add support for files where the magic
number isn't byte-order dependent (e.g., Microsoft Network Monitor), we
can handle them more cleanly (check for the standard magic number as a
4-byte array, rather than as its numerical value in both the host's byte
order and the byte-swapped byte order).

diff --git a/savefile.c b/savefile.c
index 9e4724b23fb81a22aec30dcae8e1fdafe467a00c..7adee42ffb4d1387a52408042acf8c68265b8a26 100644 (file)
--- a/savefile.c
+++ b/savefile.c
@@ -332,7 +332,7 @@ pcap_t* pcap_hopen_offline(intptr_t osfd, char *errbuf)
 }
 #endif
 
 }
 #endif
 

-static pcap_t *(*check_headers[])(bpf_u_int32, FILE *, u_int, char *, int *) = {
+static pcap_t *(*check_headers[])(const uint8_t *, FILE *, u_int, char *, int *) = {

        pcap_check_header,
        pcap_ng_check_header
 };
        pcap_check_header,
        pcap_ng_check_header
 };


@@ -347,7 +347,7 @@ pcap_fopen_offline_with_tstamp_precision(FILE *fp, u_int precision,
     char *errbuf)
 {
        register pcap_t *p;
     char *errbuf)
 {
        register pcap_t *p;

-       bpf_u_int32 magic;
+       uint8_t magic[4];

        size_t amt_read;
        u_int i;
        int err;
        size_t amt_read;
        u_int i;
        int err;


@@ -359,8 +359,8 @@ pcap_fopen_offline_with_tstamp_precision(FILE *fp, u_int precision,
         * Windows Sniffer, and Microsoft Network Monitor) all have magic
         * numbers that are unique in their first 4 bytes.
         */
         * Windows Sniffer, and Microsoft Network Monitor) all have magic
         * numbers that are unique in their first 4 bytes.
         */

-       amt_read = fread(&magic, sizeof(magic), 1, fp);
-       if (amt_read != 1) {
+       amt_read = fread(&magic, 1, sizeof(magic), fp);
+       if (amt_read != sizeof(magic)) {

                if (ferror(fp)) {
                        pcap_fmt_errmsg_for_errno(errbuf, PCAP_ERRBUF_SIZE,
                            errno, "error reading dump file");
                if (ferror(fp)) {
                        pcap_fmt_errmsg_for_errno(errbuf, PCAP_ERRBUF_SIZE,
                            errno, "error reading dump file");

diff --git a/sf-pcap.c b/sf-pcap.c
index 87fc5ce1098b914f91f155c1dae8e0e16f056c3d..f56fff745f538541c19fff9fac72679fe97e109c 100644 (file)
--- a/sf-pcap.c
+++ b/sf-pcap.c
@@ -150,9 +150,10 @@ struct pcap_sf {
  * relevant information from the header.
  */
 pcap_t *
  * relevant information from the header.
  */
 pcap_t *

-pcap_check_header(bpf_u_int32 magic, FILE *fp, u_int precision, char *errbuf,
+pcap_check_header(const uint8_t *magic, FILE *fp, u_int precision, char *errbuf,

                  int *err)
 {
                  int *err)
 {

+       bpf_u_int32 magic_int;

        struct pcap_file_header hdr;
        size_t amt_read;
        pcap_t *p;
        struct pcap_file_header hdr;
        size_t amt_read;
        pcap_t *p;


@@ -169,11 +170,14 @@ pcap_check_header(bpf_u_int32 magic, FILE *fp, u_int precision, char *errbuf,
         * number for a pcap savefile, or for a byte-swapped pcap
         * savefile.
         */
         * number for a pcap savefile, or for a byte-swapped pcap
         * savefile.
         */

-       if (magic != TCPDUMP_MAGIC && magic != KUZNETZOV_TCPDUMP_MAGIC &&
-           magic != NSEC_TCPDUMP_MAGIC) {
-               magic = SWAPLONG(magic);
-               if (magic != TCPDUMP_MAGIC && magic != KUZNETZOV_TCPDUMP_MAGIC &&
-                   magic != NSEC_TCPDUMP_MAGIC)
+       memcpy(&magic_int, magic, sizeof(magic_int));
+       if (magic_int != TCPDUMP_MAGIC &&
+           magic_int != KUZNETZOV_TCPDUMP_MAGIC &&
+           magic_int != NSEC_TCPDUMP_MAGIC) {
+               magic_int = SWAPLONG(magic_int);
+               if (magic_int != TCPDUMP_MAGIC &&
+                   magic_int != KUZNETZOV_TCPDUMP_MAGIC &&
+                   magic_int != NSEC_TCPDUMP_MAGIC)

                        return (NULL);  /* nope */
                swapped = 1;
        }
                        return (NULL);  /* nope */
                swapped = 1;
        }


@@ -182,7 +186,7 @@ pcap_check_header(bpf_u_int32 magic, FILE *fp, u_int precision, char *errbuf,
         * They are.  Put the magic number in the header, and read
         * the rest of the header.
         */
         * They are.  Put the magic number in the header, and read
         * the rest of the header.
         */

-       hdr.magic = magic;
+       hdr.magic = magic_int;

        amt_read = fread(((char *)&hdr) + sizeof hdr.magic, 1,
            sizeof(hdr) - sizeof(hdr.magic), fp);
        if (amt_read != sizeof(hdr) - sizeof(hdr.magic)) {
        amt_read = fread(((char *)&hdr) + sizeof hdr.magic, 1,
            sizeof(hdr) - sizeof(hdr.magic), fp);
        if (amt_read != sizeof(hdr) - sizeof(hdr.magic)) {


@@ -273,7 +277,7 @@ pcap_check_header(bpf_u_int32 magic, FILE *fp, u_int precision, char *errbuf,
        switch (precision) {
 
        case PCAP_TSTAMP_PRECISION_MICRO:
        switch (precision) {
 
        case PCAP_TSTAMP_PRECISION_MICRO:

-               if (magic == NSEC_TCPDUMP_MAGIC) {
+               if (magic_int == NSEC_TCPDUMP_MAGIC) {

                        /*
                         * The file has nanoseconds, the user
                         * wants microseconds; scale the
                        /*
                         * The file has nanoseconds, the user
                         * wants microseconds; scale the


@@ -290,7 +294,7 @@ pcap_check_header(bpf_u_int32 magic, FILE *fp, u_int precision, char *errbuf,
                break;
 
        case PCAP_TSTAMP_PRECISION_NANO:
                break;
 
        case PCAP_TSTAMP_PRECISION_NANO:

-               if (magic == NSEC_TCPDUMP_MAGIC) {
+               if (magic_int == NSEC_TCPDUMP_MAGIC) {

                        /*
                         * The file has nanoseconds, the
                         * user wants nanoseconds; nothing to do.
                        /*
                         * The file has nanoseconds, the
                         * user wants nanoseconds; nothing to do.


@@ -344,7 +348,7 @@ pcap_check_header(bpf_u_int32 magic, FILE *fp, u_int precision, char *errbuf,
                break;
        }
 
                break;
        }
 

-       if (magic == KUZNETZOV_TCPDUMP_MAGIC) {
+       if (magic_int == KUZNETZOV_TCPDUMP_MAGIC) {

                /*
                 * XXX - the patch that's in some versions of libpcap
                 * changes the packet header but not the magic number,
                /*
                 * XXX - the patch that's in some versions of libpcap
                 * changes the packet header but not the magic number,

diff --git a/sf-pcap.h b/sf-pcap.h
index e9c7eafa655d2823f5b2a658553c81a55bf0a0a4..bc7150f4f4fc8dbc3409dba15f120345914abfa4 100644 (file)
--- a/sf-pcap.h
+++ b/sf-pcap.h
@@ -31,7 +31,7 @@
 #ifndef sf_pcap_h
 #define        sf_pcap_h
 
 #ifndef sf_pcap_h
 #define        sf_pcap_h
 

-extern pcap_t *pcap_check_header(bpf_u_int32 magic, FILE *fp,
+extern pcap_t *pcap_check_header(const uint8_t *magic, FILE *fp,

     u_int precision, char *errbuf, int *err);
 
 #endif
     u_int precision, char *errbuf, int *err);
 
 #endif

diff --git a/sf-pcapng.c b/sf-pcapng.c
index 7394ce460e67bb9e7b56c45f7cee6a4ef2d41739..52f795f76db4cf4730598d4d5ba23708d9aa1568 100644 (file)
--- a/sf-pcapng.c
+++ b/sf-pcapng.c
@@ -761,9 +761,10 @@ add_interface(pcap_t *p, struct block_cursor *cursor, char *errbuf)
  * relevant information from the header.
  */
 pcap_t *
  * relevant information from the header.
  */
 pcap_t *

-pcap_ng_check_header(bpf_u_int32 magic, FILE *fp, u_int precision, char *errbuf,
-    int *err)
+pcap_ng_check_header(const uint8_t *magic, FILE *fp, u_int precision,
+    char *errbuf, int *err)

 {
 {

+       bpf_u_int32 magic_int;

        size_t amt_read;
        bpf_u_int32 total_length;
        bpf_u_int32 byte_order_magic;
        size_t amt_read;
        bpf_u_int32 total_length;
        bpf_u_int32 byte_order_magic;


@@ -785,7 +786,8 @@ pcap_ng_check_header(bpf_u_int32 magic, FILE *fp, u_int precision, char *errbuf,
         * Check whether the first 4 bytes of the file are the block
         * type for a pcapng savefile.
         */
         * Check whether the first 4 bytes of the file are the block
         * type for a pcapng savefile.
         */

-       if (magic != BT_SHB) {
+       memcpy(&magic_int, magic, sizeof(magic_int));
+       if (magic_int != BT_SHB) {

                /*
                 * XXX - check whether this looks like what the block
                 * type would be after being munged by mapping between
                /*
                 * XXX - check whether this looks like what the block
                 * type would be after being munged by mapping between


@@ -944,12 +946,12 @@ pcap_ng_check_header(bpf_u_int32 magic, FILE *fp, u_int precision, char *errbuf,
         */
        bhdrp = (struct block_header *)p->buffer;
        shbp = (struct section_header_block *)((u_char *)p->buffer + sizeof(struct block_header));
         */
        bhdrp = (struct block_header *)p->buffer;
        shbp = (struct section_header_block *)((u_char *)p->buffer + sizeof(struct block_header));

-       bhdrp->block_type = magic;
+       bhdrp->block_type = magic_int;

        bhdrp->total_length = total_length;
        shbp->byte_order_magic = byte_order_magic;
        if (read_bytes(fp,
        bhdrp->total_length = total_length;
        shbp->byte_order_magic = byte_order_magic;
        if (read_bytes(fp,

-           (u_char *)p->buffer + (sizeof(magic) + sizeof(total_length) + sizeof(byte_order_magic)),
-           total_length - (sizeof(magic) + sizeof(total_length) + sizeof(byte_order_magic)),
+           (u_char *)p->buffer + (sizeof(magic_int) + sizeof(total_length) + sizeof(byte_order_magic)),
+           total_length - (sizeof(magic_int) + sizeof(total_length) + sizeof(byte_order_magic)),

            1, errbuf) == -1)
                goto fail;
 
            1, errbuf) == -1)
                goto fail;
 

diff --git a/sf-pcapng.h b/sf-pcapng.h
index d99b0d4e4e3e8a978ab020adddfd016e7714a4c6..835082a509893e663f07d6fc7540602319a0af23 100644 (file)
--- a/sf-pcapng.h
+++ b/sf-pcapng.h
@@ -26,7 +26,7 @@
 #ifndef sf_pcapng_h
 #define        sf_pcapng_h
 
 #ifndef sf_pcapng_h
 #define        sf_pcapng_h
 

-extern pcap_t *pcap_ng_check_header(bpf_u_int32 magic, FILE *fp,
+extern pcap_t *pcap_ng_check_header(const uint8_t *magic, FILE *fp,

     u_int precision, char *errbuf, int *err);
 
 #endif
     u_int precision, char *errbuf, int *err);
 
 #endif