signalapp/ring
signalapp/ring
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Make symbol-prefixing work on 32-bit x86.

Browse Source 
On Linux, this introduces yet another symbol to blacklist.

Change-Id: Ieafe45a25f3b41da6c6934dd9488f4ee400bcab9
Reviewed-on: https://boringssl-review.googlesource.com/c/33350
Commit-Queue: David Benjamin <davidben@google.com>
Reviewed-by: Adam Langley <agl@google.com>
This commit is contained in:
David Benjamin 2018-11-25 20:21:38 -06:00 committed by CQ bot account: commit-bot@chromium.org
parent 8c23d3a5df
commit e157dc9208
2 changed files with 16 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

17
crypto/perlasm/x86asm.pl
View File

@ -262,7 +262,20 @@ $comment This file is generated from a similarly-named Perl script in the Boring
$comment source tree. Do not edit by hand.
___
print "#if defined(__i386__)\n" unless ($win32 || $netware);
if ($win32 || $netware) {
print <<___ unless $masm;
%ifdef BORINGSSL_PREFIX
%include "boringssl_prefix_symbols_nasm.inc"
%endif
___
} else {
print <<___;
#if defined(__i386__)
#if defined(BORINGSSL_PREFIX)
#include <boringssl_prefix_symbols_asm.h>
#endif
___
}
print @out;
print "#endif\n" unless ($win32 || $netware);
}
@ -288,7 +301,7 @@ sub ::asm_init
#elsif (($type eq "nw-mwasm"))
#{ $netware=1; $mwerks=1; require "x86nasm.pl"; }
elsif (($type eq "win32"))
{ $win32=1; require "x86masm.pl"; }
{ $win32=1; $masm=1; require "x86masm.pl"; }
elsif (($type eq "macosx"))
{ $aout=1; $macosx=1; require "x86gas.pl"; }
elsif (($type eq "android"))

2
util/read_symbols.go
View File

@ -140,7 +140,7 @@ func main() {
break
}
}
if skip || isCXXSymbol(s) || strings.HasPrefix(s, "__real@") {
if skip || isCXXSymbol(s) || strings.HasPrefix(s, "__real@") || strings.HasPrefix(s, "__x86.get_pc_thunk.") {
continue
}
if _, err := fmt.Fprintln(out, s); err != nil {