<br><br><div class="gmail_quote">On Fri, Jan 27, 2012 at 7:58 AM, Alexander Potapenko <span dir="ltr"><<a href="mailto:glider@google.com">glider@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Kostya,<br>
<br>
it seems to me that this patch broke the support for ObjC.<br></blockquote><div>It is very unlikely that this patch broke the objc support in asan. </div><div>btw, can we have objc tests on linux? </div><div><br></div><div>
--kcc </div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
$ gobjdump -D bin_darwin/asan_test32<br>
...<br>
<br>
0013b460 <+[LoadSomething load]>:<br>
  13b460:       55                      push   %ebp<br>
  13b461:       89 e5                   mov    %esp,%ebp<br>
  13b463:       53                      push   %ebx<br>
  13b464:       57                      push   %edi<br>
  13b465:       56                      push   %esi<br>
  13b466:       83 ec 0c                sub    $0xc,%esp<br>
  13b469:       e8 00 00 00 00          call   13b46e <+[LoadSomething<br>
load]+0xe><br>
  13b46e:       5b                      pop    %ebx<br>
  13b46f:       8d b3 72 75 1b 00       lea    0x1b7572(%ebx),%esi<br>
  13b475:       89 34 24                mov    %esi,(%esp)<br>
  13b478:       e8 4f 77 17 00          call   2b2bcc <_strlen$stub><br>
  13b47d:       85 c0                   test   %eax,%eax<br>
  13b47f:       74 22                   je     13b4a3 <+[LoadSomething<br>
load]+0x43><br>
  13b481:       31 ff                   xor    %edi,%edi<br>
  13b483:       8d 9b 72 75 1b 00       lea    0x1b7572(%ebx),%ebx<br>
  13b489:       0f 1f 80 00 00 00 00    nopl   0x0(%eax)<br>
  13b490:       8a 06                   mov    (%esi),%al<br>
  13b492:       88 45 f3                mov    %al,-0xd(%ebp)<br>
  13b495:       89 1c 24                mov    %ebx,(%esp)<br>
  13b498:       46                      inc    %esi<br>
  13b499:       47                      inc    %edi<br>
  13b49a:       e8 2d 77 17 00          call   2b2bcc <_strlen$stub><br>
  13b49f:       39 c7                   cmp    %eax,%edi<br>
  13b4a1:       72 ed                   jb     13b490 <+[LoadSomething<br>
load]+0x30><br>
  13b4a3:       83 c4 0c                add    $0xc,%esp<br>
  13b4a6:       5e                      pop    %esi<br>
  13b4a7:       5f                      pop    %edi<br>
  13b4a8:       5b                      pop    %ebx<br>
  13b4a9:       5d                      pop    %ebp<br>
  13b4aa:       c3                      ret<br>
  13b4ab:       0f 1f 44 00 00          nopl   0x0(%eax,%eax,1)<br>
...<br>
<br>
-- as you can see, __asan_init is not called at the beginning of the function.<br>
This leads to crashes in Chromium, because some instrumented code ends<br>
up being called from such load methods before __asan_init.<br>
I'm going to take a look, but it'll be great if you know how to fix<br>
this off the top of your head.<br>
</blockquote></div><br>