diff options
author | brucedawson <brucedawson@chromium.org> | 2015-10-06 12:22:00 -0700 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2015-10-06 19:22:59 +0000 |
commit | 5604a11d546e02859db5dc75af72ce27bc14c158 (patch) | |
tree | 4af3fe43eaec732842b4999cce507c37edb1a94c /ipc/handle_win.cc | |
parent | a45aa1efa4b5e8307e9cfe119892899c3f922852 (diff) | |
download | chromium_src-5604a11d546e02859db5dc75af72ce27bc14c158.zip chromium_src-5604a11d546e02859db5dc75af72ce27bc14c158.tar.gz chromium_src-5604a11d546e02859db5dc75af72ce27bc14c158.tar.bz2 |
Detecting and fixing stringprintf.h format bugs
The print functions in stringprintf.h were not annotated for /analyze so
13 Windows specific format-string bugs accumulated. This annotates the
functions so that the /analyze builder will find the problems and fixes
the bugs.
R=thestig@chromium.org,wfh@chromium.org,jam@chromium.org
Skipping wstring presubmit checks - no new wstring usage is added
NOPRESUBMIT=true
BUG=427616
Review URL: https://codereview.chromium.org/1372153002
Cr-Commit-Position: refs/heads/master@{#352659}
Diffstat (limited to 'ipc/handle_win.cc')
-rw-r--r-- | ipc/handle_win.cc | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/ipc/handle_win.cc b/ipc/handle_win.cc index 5d25905..51f3f95 100644 --- a/ipc/handle_win.cc +++ b/ipc/handle_win.cc @@ -48,7 +48,7 @@ bool ParamTraits<HandleWin>::Read(const Message* m, // static void ParamTraits<HandleWin>::Log(const param_type& p, std::string* l) { - l->append(base::StringPrintf("0x%X", p.get_handle())); + l->append(base::StringPrintf("0x%p", p.get_handle())); l->append(base::IntToString(p.get_permissions())); } |