BIND 10 master, updated. 5ba017624caefa28eb330dbcf4bafe0890fbb899 [master] Add cppcheck suppressions for sscanf() false-positives

BIND 10 source code commits bind10-changes at lists.isc.org
Tue Feb 18 15:18:54 UTC 2014


The branch, master has been updated
       via  5ba017624caefa28eb330dbcf4bafe0890fbb899 (commit)
      from  fe4183783d075bd2ac2aef3d7dc6e3de999d8589 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
commit 5ba017624caefa28eb330dbcf4bafe0890fbb899
Author: Mukund Sivaraman <muks at isc.org>
Date:   Tue Feb 18 19:34:49 2014 +0530

    [master] Add cppcheck suppressions for sscanf() false-positives
    
    Reviewed by Thomas Markwalder on Jabber.

-----------------------------------------------------------------------

Summary of changes:
 src/lib/dns/master_loader.cc |    2 ++
 1 file changed, 2 insertions(+)

-----------------------------------------------------------------------
diff --git a/src/lib/dns/master_loader.cc b/src/lib/dns/master_loader.cc
index 431c34b..80b1053 100644
--- a/src/lib/dns/master_loader.cc
+++ b/src/lib/dns/master_loader.cc
@@ -671,6 +671,7 @@ MasterLoader::MasterLoaderImpl::generateForIter(const std::string& str,
               int offset = 0;
               unsigned int width;
               char base[2] = {'d', 0}; // char plus null byte
+              // cppcheck-suppress invalidscanf
               const int n = sscanf(scan_str, "{%d,%u,%1[doxXnN]}",
                                    &offset, &width, base);
               switch (n) {
@@ -807,6 +808,7 @@ MasterLoader::MasterLoaderImpl::doGenerate() {
     unsigned int start;
     unsigned int stop;
     unsigned int step;
+    // cppcheck-suppress invalidscanf
     const int n = sscanf(range.c_str(), "%u-%u/%u", &start, &stop, &step);
     if ((n < 2) || (stop < start)) {
         reportError(lexer_.getSourceName(), lexer_.getSourceLine(),



More information about the bind10-changes mailing list