StarMirror/CppCoreGuidelines
1
0
Fork 0
mirror of https://github.com/isocpp/CppCoreGuidelines.git synced 2024-03-22 13:30:58 +08:00
Code Issues Projects Releases Wiki Activity

use sizeof(buffer) instead of MAX

Browse Source
This commit is contained in:
Chris Guzak 2018-03-13 15:28:26 -07:00 committed by GitHub
parent 09035eb1e4
commit 373f4c4808
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
CppCoreGuidelines.md
View File

@ -1584,7 +1584,7 @@ Consider a famous security bug:
{ {
char buffer[MAX]; char buffer[MAX];
// ... // ...
memset(buffer, 0, MAX); memset(buffer, 0, sizeof(buffer));
} }
There was no postcondition stating that the buffer should be cleared and the optimizer eliminated the apparently redundant `memset()` call: There was no postcondition stating that the buffer should be cleared and the optimizer eliminated the apparently redundant `memset()` call:
@ -1593,7 +1593,7 @@ There was no postcondition stating that the buffer should be cleared and the opt
{ {
char buffer[MAX]; char buffer[MAX];
// ... // ...
memset(buffer, 0, MAX); memset(buffer, 0, sizeof(buffer));
Ensures(buffer[0] == 0); Ensures(buffer[0] == 0);
} }