summaryrefslogtreecommitdiff
path: root/serializable.cpp
diff options
context:
space:
mode:
authorShawn Willden <swillden@google.com>2014-12-08 12:41:59 -0700
committerShawn Willden <swillden@google.com>2015-01-22 16:35:35 -0700
commit1834d5f82a7ad5884c184fd22c702ac9d915af45 (patch)
treeeb018bc40f4f546748a297a96522b2fd2d3283be /serializable.cpp
parentb7361134bff4d3d7ef1d5a3c60e50c9952dc2b56 (diff)
downloadkeymaster-1834d5f82a7ad5884c184fd22c702ac9d915af45.tar.gz
Remove redundant NULL checks on calls to memset_s.
This change was already reviewed, merged and reverted, so I'm skipping the review step this time. Change-Id: I6f7ecb71fc03b9a821c81e1bc0f54952225d9da8
Diffstat (limited to 'serializable.cpp')
-rw-r--r--serializable.cpp3
1 files changed, 1 insertions, 2 deletions
diff --git a/serializable.cpp b/serializable.cpp
index 9cfbac7..d0917da 100644
--- a/serializable.cpp
+++ b/serializable.cpp
@@ -131,8 +131,7 @@ bool Buffer::Deserialize(const uint8_t** buf_ptr, const uint8_t* end) {
}
void Buffer::Clear() {
- if (buffer_.get())
- memset_s(buffer_.get(), 0, buffer_size_);
+ memset_s(buffer_.get(), 0, buffer_size_);
buffer_.reset();
read_position_ = 0;
write_position_ = 0;