Looks good.

On Aug 23, 2013, at 10:10 AM, Virgile Bello <[email protected]> wrote:

> File offset is not set properly in PECOFF.
> 
> http://llvm-reviews.chandlerc.com/D1488
> 
> Files:
>  source/Plugins/ObjectFile/PECOFF/ObjectFilePECOFF.cpp
> 
> Index: source/Plugins/ObjectFile/PECOFF/ObjectFilePECOFF.cpp
> ===================================================================
> --- source/Plugins/ObjectFile/PECOFF/ObjectFilePECOFF.cpp
> +++ source/Plugins/ObjectFile/PECOFF/ObjectFilePECOFF.cpp
> @@ -434,6 +434,8 @@
>                     m_coff_header_opt.data_dirs[i].vmaddr = 
> m_data.GetU32(offset_ptr);
>                     m_coff_header_opt.data_dirs[i].vmsize = 
> m_data.GetU32(offset_ptr);
>                 }
> +
> +                m_file_offset = m_coff_header_opt.image_base;
>             }
>         }
>     }
> <D1488.1.patch>_______________________________________________
> lldb-commits mailing list
> [email protected]
> http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits

_______________________________________________
lldb-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits

Reply via email to