Browse code

Fix newpe merge bug in cli_peheader (header32 was read twice)

git-svn: trunk@2523

aCaB authored on 2006/11/28 00:54:00
Showing 2 changed files
... ...
@@ -1,3 +1,7 @@
1
+Mon Nov 27 16:51:43 CET 2006 (acab)
2
+-----------------------------------
3
+  * libclamav/pe.c: Fix merge bug in new PE model
4
+
1 5
 Mon Nov 27 11:37:18 CET 2006 (tk)
2 6
 ---------------------------------
3 7
   * libclamav/pe.c: use unaligned section's virtual size in Magistr detection
... ...
@@ -2547,12 +2547,6 @@ int cli_peheader(int desc, struct cli_exe_info *peinfo)
2547 2547
 
2548 2548
     if(!pe_plus) { /* PE */
2549 2549
 	cli_dbgmsg("File format: PE\n");
2550
-
2551
-	if(cli_readn(desc, &optional_hdr32, sizeof(struct pe_image_optional_hdr32)) != sizeof(struct pe_image_optional_hdr32)) {
2552
-	    cli_dbgmsg("Can't read optional file header\n");
2553
-	    return -1;
2554
-	}
2555
-
2556 2550
 	if (EC16(file_hdr.SizeOfOptionalHeader)!=sizeof(struct pe_image_optional_hdr32)) {
2557 2551
 	    /* Seek to the end of the long header */
2558 2552
 	    lseek(desc, (EC16(file_hdr.SizeOfOptionalHeader)-sizeof(struct pe_image_optional_hdr32)), SEEK_CUR);