google / sanitizers

AddressSanitizer, ThreadSanitizer, MemorySanitizer
Other
11.43k stars 1.03k forks source link

Assembly instrumentation: handle more instructions #282

Open ramosian-glider opened 9 years ago

ramosian-glider commented 9 years ago

Originally reported on Google Code with ID 282

On X86, almost any instruction can have a memory argument. At the moment we instrument
only a minority of them.

We could use MCInstrInfo/MCRegisterInfo to guess instruction memory properties.

Reported by eugenis@google.com on 2014-03-24 07:31:26

ramosian-glider commented 9 years ago
Attached (very hackish) patch almost passes ffmpeg.
- it passes TargetMachine through global var, just because
- mishandles movzbl and, in general, determines memory access size from the size of
the other argument. It's wrong for a lot of instructions.

I may work on it some more later.

Reported by eugenis@google.com on 2014-03-24 07:37:30


ramosian-glider commented 9 years ago

Reported by ramosian.glider on 2015-07-30 09:05:32

ramosian-glider commented 9 years ago
Adding Project:AddressSanitizer as part of GitHub migration.

Reported by ramosian.glider on 2015-07-30 09:06:56

morehouse commented 6 years ago

@eugenis: Status?