Fixed PE section sanity check derp.

Accidentally checking for bounds of offset + offset instead of offset + size.
0xnobody · Aug 21, 2020 · 30cba9b · 30cba9b
1 parent 04fa2b5
commit 30cba9b
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/VMPAttack/vmpattack.cpp b/VMPAttack/vmpattack.cpp
@@ -143,7 +143,7 @@ namespace vmpattack
         {
             // Sanity check for potentially broken PEs.
             //
-            if ( image.raw_bytes.size() >= section.physical_address + section.physical_address )
+            if ( image.raw_bytes.size() >= section.physical_address + section.physical_size )
                 memcpy( &mapped_buffer[ section.virtual_address ], &image.raw_bytes[ section.physical_address ], section.physical_size );
         }
 

diff --git a/VMPAttack_Tester/VMPAttack_Tester.cpp b/VMPAttack_Tester/VMPAttack_Tester.cpp
@@ -21,9 +21,9 @@ struct much_complex_object
 };
 #pragma pack(pop)
 
-int main()
+int main( int argc, const char* args[] )
 {
-	char r = getchar();
+
 
 	while ( r >= 0x50 )
 	{