Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(94)

Issue 226640043: Add a basic runtime class with logging and DllMain intercept to syzyasan_hp.dll. (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
9 years ago by cseri
Modified:
8 years, 12 months ago
CC:
syzygy-changes_googlegroups.com
Base URL:
https://code.google.com/p/syzygy.git@master
Visibility:
Public.

Description

Add a basic runtime class with logging and DllMain intercept to syzyasan_hp.dll. BUG= R=chrisha@chromium.org Committed: https://code.google.com/p/syzygy/source/detail?r=3a684929ec99bf44d621b1e3fcdc81609c354f8f

Patch Set 1 #

Total comments: 2

Patch Set 2 : Made the runtime a proper singleton and added a test to check if it's loaded by the dll. #

Total comments: 4

Patch Set 3 : Fix nits. #

Unified diffs Side-by-side diffs Delta from patch set Stats (+482 lines, -2 lines) Patch
M syzygy/agent/asan/asan.gyp View 1 5 chunks +61 lines, -2 lines 0 comments Download
A syzygy/agent/asan/hot_patching_asan_runtime.h View 1 1 chunk +109 lines, -0 lines 0 comments Download
A syzygy/agent/asan/hot_patching_asan_runtime.cc View 1 2 1 chunk +110 lines, -0 lines 0 comments Download
A syzygy/agent/asan/hot_patching_asan_runtime_unittest.cc View 1 2 1 chunk +152 lines, -0 lines 0 comments Download
M syzygy/agent/asan/syzyasan_hp.cc View 1 2 chunks +44 lines, -0 lines 0 comments Download
M syzygy/agent/asan/syzyasan_hp.def View 1 1 chunk +6 lines, -0 lines 0 comments Download

Messages

Total messages: 6
cseri
Simple runtime class for the hot patching library. Of course it cannot hot patch yet ...
9 years ago (2015-04-24 04:08:12 UTC) #1
chrisha
https://codereview.appspot.com/226640043/diff/1/syzygy/agent/asan/hot_patching_asan_runtime.h File syzygy/agent/asan/hot_patching_asan_runtime.h (right): https://codereview.appspot.com/226640043/diff/1/syzygy/agent/asan/hot_patching_asan_runtime.h#newcode38 syzygy/agent/asan/hot_patching_asan_runtime.h:38: // Constructs a the class and initialized the static ...
9 years ago (2015-04-27 15:28:19 UTC) #2
cseri
Made the runtime a proper singleton and added a test to check if it's loaded ...
8 years, 12 months ago (2015-05-02 00:31:59 UTC) #3
chrisha
Awesome, this lgtm! https://codereview.appspot.com/226640043/diff/20001/syzygy/agent/asan/hot_patching_asan_runtime.cc File syzygy/agent/asan/hot_patching_asan_runtime.cc (right): https://codereview.appspot.com/226640043/diff/20001/syzygy/agent/asan/hot_patching_asan_runtime.cc#newcode33 syzygy/agent/asan/hot_patching_asan_runtime.cc:33: std::to_string(reinterpret_cast<int>(instance)) + Indent these lines another ...
8 years, 12 months ago (2015-05-04 13:35:20 UTC) #4
cseri
Thanks, committing! https://codereview.appspot.com/226640043/diff/20001/syzygy/agent/asan/hot_patching_asan_runtime.cc File syzygy/agent/asan/hot_patching_asan_runtime.cc (right): https://codereview.appspot.com/226640043/diff/20001/syzygy/agent/asan/hot_patching_asan_runtime.cc#newcode33 syzygy/agent/asan/hot_patching_asan_runtime.cc:33: std::to_string(reinterpret_cast<int>(instance)) + On 2015/05/04 13:35:20, chrisha wrote: ...
8 years, 12 months ago (2015-05-04 19:05:58 UTC) #5
cseri
8 years, 12 months ago (2015-05-04 19:18:19 UTC) #6
Message was sent while issue was closed.
Committed patchset #3 (id:40001) manually as
3a684929ec99bf44d621b1e3fcdc81609c354f8f (presubmit successful).
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b