net-snmp 3.9.0 → 3.9.2

package/README.md

@@ -3243,6 +3243,14 @@ Example programs are included under the module's `example` directory.
 
  * Add ProcessingError to handle agent/receiver decode failures
 
+## Version 3.9.1 - 16/03/2023
+
+ * Fix MIB parsing of unclosed brackets in comments
+
+## Version 3.9.2 - 26/04/2023
+
+ * Fix MIB parsing of non-comments in descriptions and unmatched quotations
+
 # License
 
 Copyright (c) 2020 Mark Abrahams <mark@abrahams.co.nz>

package/example/mib-parser.js

@@ -35,12 +35,26 @@ var mib = agent.getMib ();
 var authorizer = agent.getAuthorizer ();
 authorizer.addCommunity ("public");
 
+
 // IF-MIB load and providers registration
 store.loadFromFile (mibDir + "IANAifType-MIB.mib");
 store.loadFromFile (mibDir + "IF-MIB.mib");
 providers = store.getProvidersForModule ("IF-MIB");
 mib.registerProviders (providers);
 
+store.loadFromFile (mibDir + "SNMP-FRAMEWORK-MIB.mib");
+store.loadFromFile (mibDir + "INET-ADDRESS-MIB.mib");
+store.loadFromFile (mibDir + "CISCO-SMI.mib");
+store.loadFromFile (mibDir + "CISCO-TC.mib");
+store.loadFromFile (mibDir + "CISCO-LWAPP-TC-MIB.mib");
+store.loadFromFile (mibDir + "CISCO-LWAPP-WLAN-MIB.mib");
+store.loadFromFile (mibDir + "ENTITY-MIB.mib");
+store.loadFromFile (mibDir + "CISCO-LWAPP-RF-MIB.mib");
+store.loadFromFile (mibDir + "CISCO-LWAPP-AP-MIB.mib");
+store.loadFromFile (mibDir + "CISCO-LWAPP-DOT11-MIB.mib");
+store.loadFromFile (mibDir + "CISCO-LWAPP-REAP-MIB.mib");
+store.loadFromFile (mibDir + "CISCO-LWAPP-CDP-MIB.mib");
+
 // ifNumber
 // Scalar type - setScalarValue() and getScalarValue() are the entire API for these
 mib.setScalarValue ("ifNumber", 5);

package/lib/mib.js

@@ -1,18 +1,9 @@
 var fs = require('fs');
 
 var MIB = function (dir) {
-    return ({
-        directory: dir ? dir : '',
-        SymbolBuffer: {},
-        StringBuffer: '',
-        Modules: {},
-        Objects: {},
-        MACROS: [],
-        CurrentObject: null,
-        TempObject: {},
-        CurrentClause: '',
-        WaitFor: '',
-        CharBuffer: {
+
+    var initializeBuffer = function (buffer) {
+        return Object.assign(buffer, {
             logit: false,
             lastChar: '',
             state: '',
@@ -27,11 +18,26 @@ var MIB = function (dir) {
             inComment: false,
             inGroup: 0,
             builder: '',
-            Table: {},
             ColumnIndex: 0,
             RowIndex: 0,
+            PreviousRow: 0
+        });
+    };
+
+    var newMIB = ({
+        directory: dir ? dir : '',
+        SymbolBuffer: {},
+        StringBuffer: '',
+        Modules: {},
+        Objects: {},
+        MACROS: [],
+        CurrentObject: null,
+        TempObject: {},
+        CurrentClause: '',
+        WaitFor: '',
+        CharBuffer: {
+            Table: {},
             ModuleName: {},
-            PreviousRow: 0,
             Append: function (char) {
                 this.builder += char;
             },
@@ -45,12 +51,7 @@ var MIB = function (dir) {
                 this.builder.length = 0;
                 if (!this.Table[FileName]) {
                     this.Table[FileName] = [];
-                }
-                if (row == 0) {
-                    this.RowIndex = 0;
-                    this.PreviousRow = 0;
-                }
-                if (this.PreviousRow < row) {
+                } else if (this.PreviousRow < row) {
                     this.RowIndex++;
                     this.ColumnIndex = 0;
                     this.PreviousRow = row;
@@ -60,7 +61,10 @@ var MIB = function (dir) {
                 var C = this.ColumnIndex;
 
                 if (!this.Table[FileName][R]) {
-                    this.Table[FileName][R] = [];
+                    this.Table[FileName][R] = Object.defineProperty([], "line", {
+                        enumerable: false,
+                        value: row + 1
+                    });
                 }
                 this.isEqual = false;
                 switch (symbol) {
@@ -128,8 +132,7 @@ var MIB = function (dir) {
             this.ParseModule(FileName.split('/')[FileName.split('/').length - 1].split('.')[0], fs.readFileSync(FileName).toString());
         },
         ParseModule: function (FileName, Contents) {
-            this.CharBuffer.RowIndex = 0;
-            this.CharBuffer.ColumnIndex = 0;
+            initializeBuffer(this.CharBuffer);
 
             var lines = Contents.split('\n');
             var line = '';
@@ -159,10 +162,10 @@ var MIB = function (dir) {
                     }
                     break;
                 case '{':
-                    if (this.CharBuffer.isEqual) { this.CharBuffer.isOID = true; }
+                    if ( ! this.CharBuffer.isComment && this.CharBuffer.isEqual ) { this.CharBuffer.isOID = true; }
                 case '[':
                 case '(':
-                    if ( ! this.CharBuffer.isString ) {
+                    if ( ! this.CharBuffer.isComment && ! this.CharBuffer.isString ) {
                         this.CharBuffer.nested++;
                         if ( char == '(') {
                             this.CharBuffer.inGroup++;
@@ -182,7 +185,7 @@ var MIB = function (dir) {
                 case '}':
                 case ']':
                 case ')':
-                    if ( ! this.CharBuffer.isString ) {
+                    if ( ! this.CharBuffer.isComment && ! this.CharBuffer.isString ) {
                         this.CharBuffer.nested--;
                         if (this.CharBuffer.nested < 0) {
                             this.CharBuffer.nested = 0;
@@ -234,7 +237,7 @@ var MIB = function (dir) {
                     break;
                 case "-":
                     this.CharBuffer.Append(char);
-                    if (this.CharBuffer.lastChar == '-') {
+                    if (!this.CharBuffer.isString && this.CharBuffer.lastChar == '-') {
                         this.CharBuffer.isComment = true;
                         this.CharBuffer.builder = this.CharBuffer.builder.split('--')[0];
                         this.CharBuffer.Fill(FileName, row, column);
@@ -244,7 +247,7 @@ var MIB = function (dir) {
                     break;
                 case '"':
                     if (this.CharBuffer.isComment && !this.CharBuffer.isString && !this.CharBuffer.inComment) {
-                        //011 = COMMENT 
+                        //011 = COMMENT
                         //IF 011 SET 101
                         this.CharBuffer.isComment = true;
                         this.CharBuffer.isString = false;
@@ -289,18 +292,42 @@ var MIB = function (dir) {
             for (var FileName in Table) {
                 ModuleName = this.CharBuffer.ModuleName[FileName];
                 this.SymbolBuffer[ModuleName] = [];
-                for (var r = 0; r < Table[FileName].length; r++) {
-                    for (var c = 0; c < Table[FileName][r].length; c++) {
-                        var symbol = Table[FileName][r][c];
+                var foundTheEnd = false;
+                var lastGoodDeclaration = [ 'none' ];
+                var file = Table[FileName];
+                for (var r = 0; r < file.length; r++) {
+                    var row = file[r];
+                    for (var c = 0; c < row.length; c++) {
+                        var symbol = row[c];
+                        var addSymbol = true;
                         switch (symbol) {
+                            case 'END':
+                                foundTheEnd = true;
+                                break;
+                            case '::=':
+                                foundTheEnd = false;
+                                lastGoodDeclaration = row;
+                                break;
                             default:
-                                if (symbol.indexOf('--') != 0) {//REMOVE COMMENTS
+                                if (symbol.indexOf('--') == 0) {//REMOVE COMMENTS
                                     //console.log(ModuleName, symbol);
-                                    this.SymbolBuffer[ModuleName].push(symbol);
+                                    addSymbol = false;
+                                } else {
+                                    foundTheEnd = false;
                                 }
                         }
+                        if (addSymbol) {
+                            this.SymbolBuffer[ModuleName].push(symbol);
+                        }
                     }
                 }
+                if (!foundTheEnd) {
+                    // Warn that the contents are malformed
+                    console.warn(
+                        '[%s]: Incorrect formatting: no END statement found - last good declaration "%s" (line %s)',
+                        ModuleName, lastGoodDeclaration.join(' '), lastGoodDeclaration.line
+                    );
+                }
 
             }
             this.Compile();
@@ -359,7 +386,7 @@ var MIB = function (dir) {
                                         //BUILD OBJECT FROM MACRO TYPE NOTATION
                                         var MARCO = this[Symbols[r]];
                                         if (!MARCO) {
-                                            //HACK IF MARCO IS NOT FOUND 
+                                            //HACK IF MARCO IS NOT FOUND
                                             //MARCO = {};
                                             //return;
                                         }
@@ -401,7 +428,7 @@ var MIB = function (dir) {
                                                         val = val.replace("{", "").replace("}", "").split(",");
                                                     }
                                                 }
-                                               
+
                                                 switch (key) {
                                                     case 'SYNTAX':
                                                         switch (val) {
@@ -727,6 +754,11 @@ var MIB = function (dir) {
             }
         }
     });
+
+    // Complete buffer setup before returning to caller.
+    initializeBuffer(newMIB.CharBuffer);
+
+    return newMIB;
 };
 
 module.exports = exports = MIB;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "net-snmp",
-  "version": "3.9.0",
+  "version": "3.9.2",
   "description": "JavaScript implementation of the Simple Network Management Protocol (SNMP)",
   "main": "index.js",
   "directories": {