Libsmi April 2008

libsmi@ibr.cs.tu-bs.de

5 participants
18 discussions

libsmi numeric constraints convertion bug
by Ilya Etingof 24 Nov '08

24 Nov '08

I've run into a possible problem in SVN version of libsmi. I have not done any debugging, so this is just a bug report: When I do: smidump -f python TCP-MIB.txt for the following MIB text: tcpMaxConn OBJECT-TYPE SYNTAX Integer32 (-1 | 0..2147483647) MAX-ACCESS read-only STATUS current DESCRIPTION "The limit on the total number of TCP connections the entity can support. In entities where the maximum number of connections is dynamic, this object should contain the value -1." ::= { tcp 4 } smidump generates: "tcpMaxConn" : { "nodetype" : "scalar", "moduleName" : "TCP-MIB", "oid" : "1.3.6.1.2.1.6.4", "status" : "current", "syntax" : { "type" : { "basetype" : "Integer32", "ranges" : [ { "min" : "-1", "max" : "2147483647" }, ], "range" : { "min" : "4294967295", "max" : "2147483647" }, }, }, "access" : "readonly", "description" : """ [skipped] """, }, # scalar Where range constraints seem to be broken at: 1. Range minimum (0) gets converted into 4294967295. 2. Single value constraint (-1) gets translated into a value range constraint with wildly inflated upper range (2147483647). Current stable release (0.4.5) behaves differently (yet, incorrectly) in this respect: "tcpMaxConn" : { "nodetype" : "scalar", "moduleName" : "TCP-MIB", "oid" : "1.3.6.1.2.1.6.4", "status" : "current", "syntax" : { "type" : { "basetype" : "Integer32", "range" : { "min" : "-1", "max" : "2147483647" }, }, }, "access" : "readonly", "description" : """ [skipped] """, }, # scalar Is there currently a fix for this issue? Thanks, ilya

2 17

Patches to build generated files on Windows
by Anders Broman 04 May '08

04 May '08

WARNING: This e-mail has been altered by MIMEDefang. Following this paragraph are indications of the actual changes made. For more information about your site's MIMEDefang policy, contact MIMEDefang Administrator <admin(a)ibr.cs.tu-bs.de>. For more information about MIMEDefang, see: http://www.roaringpenguin.com/mimedefang/enduser.php3 An attachment named make_errormacros.sh was removed from this document as it constituted a security hazard. If you require this document, please contact the sender and arrange an alternate means of receiving it. Hi, I have been trying to build the generated files on windows using Cygwin Flex and Bison (avoiding the autogen stuff) and I think I'm almost there The makefile patch can be checked in if you think this is ok, currently the makefile in ../lib is not called from there. Make_errormacros does not produce a vallid errormacros.h ( I don't know why I copied it from makefile.in) The result is #define { 0, ERR_INTERNAL, "internal", 0 #define { 0, ERR_MAX_LEX_DEPTH, "import-depth", 1 : Instead of: #define ERR_INTERNAL 0 #define ERR_MAX_LEX_DEPTH 1 I think the makefile.nmake and make_errormacros.sh can be checked in in ../lib/ and continue to be worked on there. Possibly the stuff related to errormacros can be removed for the time beeing. Is this route worth persuing or are you happy with things as they are? <<make_errormacros.sh>> <<makefile.nmake>> <<makefile.patch.txt>> Regards Anders

2 3

missing ELSE
by Andrew Hood 22 Apr '08

22 Apr '08

Index: config.nmake =================================================================== --- config.nmake (revision 8127) +++ config.nmake (working copy) @@ -74,6 +74,7 @@ TOOLS32 = C:\PROGRA~1\MICROS~2\VC98 !ELSEIF "$(MSVC_VARIANT)" == "MSVC2005" TOOLS32 = "C:\Program Files\Microsoft Visual Studio 8\VC" +!ELSE !ERROR MSVC_VARIANT unknown !ENDIF -- There's no point in being grown up if you can't be childish sometimes. -- Dr. Who

2 1

dump-sming.c
by Andrew Hood 21 Apr '08

21 Apr '08

Appears to have some bugs. smiGetFirstImport/smiGetNextImport are being used in fprintImports instead of the importList that was built. The attached patch (which can probably be done better) just leaves a couple of: 170d169 < status current; diff errors in "make check" -- There's no point in being grown up if you can't be childish sometimes. -- Dr. Who Index: tools/dump-sming.c =================================================================== --- tools/dump-sming.c (revision 8107) +++ tools/dump-sming.c (working copy) @@ -696,30 +696,30 @@ static void fprintImports(FILE *f, SmiModule *smiModule) { - SmiImport *import; + Import **import; char *lastModuleName = NULL; int pos = 0, len, maxlen = 0; createImportList(smiModule); - for (import = smiGetFirstImport(smiModule); import; import = smiGetNextImport(import)) { - len = strlen(import->module); + for (import = &importList; *import; import = &(*import)->nextPtr) { + len = strlen((*import)->module); maxlen = (len > maxlen) ? len : maxlen; } - for (import = smiGetFirstImport(smiModule); import; import = smiGetNextImport(import)) { - int yaba = !lastModuleName || strcmp(import->module, lastModuleName); + for (import = &importList; *import; import = &(*import)->nextPtr) { + int yaba = !lastModuleName || strcmp((*import)->module, lastModuleName); if (yaba) { if (lastModuleName) { fprint(f, ");\n"); } fprintSegment(f, INDENT, "", 0); - fprint(f, "import %-*s (", maxlen, import->module); + fprint(f, "import %-*s (", maxlen, (*import)->module); pos = INDENT + 12 + maxlen; } else { fprint(f, ", "); } - len = strlen(import->name); + len = strlen((*import)->name); if (len + pos > INDENTMAX) { fprint(f, "\n"); fprintSegment(f, INDENT, "", 0); @@ -727,9 +727,9 @@ fprint(f, " %-*s ", maxlen, ""); pos = INDENT + 12 + maxlen; } - fprint(f, "%s", import->name); + fprint(f, "%s", (*import)->name); pos += len; - lastModuleName = import->module; + lastModuleName = (*import)->module; } if (lastModuleName) {

1 0

Config.nmake missing from distribution
by Anders Broman 21 Apr '08

21 Apr '08

Hi, win/config.nmake nnedes to be added to EXTRA_DIST in toplevel makefile.am. Regards Anders Broman

2 1

libsmi release 0.4.8
by Frank Strauß 21 Apr '08

21 Apr '08

Hi! There is a new release. It - hopefully :-) - clearifies recent confusion with inconsistent version numbers (sorry for that!), adds two new IETF MIBs, fixes numerous compiler warnings, and contains a couple of Windows related updates. As always, thanks to all the people who submitted bug reports and patches! http://www.ibr.cs.tu-bs.de/projects/libsmi/ ftp://ftp.ibr.cs.tu-bs.de/pub/local/libsmi/ Enjoy, -frank

2 2

Patch to set tools dir pepending on MSVC version
by Anders Broman 19 Apr '08

19 Apr '08

Hi, Patch to automatically sett the TOOLS32 lib depending on the MSVC version used and explain a bit what to set. <<config.nmake.txt>> Regards Anders

3 3

Libsmi version?
by Anders Broman 18 Apr '08

18 Apr '08

Hi, Shouldn't LIBSMI_PATCHLEVEL=6 be LIBSMI_PATCHLEVEL=7 in configure.in ? Regards Anders

3 2

Typo in config.h
by Anders Broman 18 Apr '08

18 Apr '08

Hi, Sorry I made a typo in config.h I think. Regards Anders Index: config.h =================================================================== --- config.h (revision 8082) +++ config.h (working copy) @@ -1,5 +1,5 @@ /* - * config.h. Generated manually for windows / vc++ 6.0 */ + * config.h. Generated manually for windows / vc++ 6.0 * * @(#) $Id$ */

1 0

Id marking for config.h
by Anders Broman 17 Apr '08

17 Apr '08

Index: config.h =================================================================== --- config.h (revision 8072) +++ config.h (working copy) @@ -1,4 +1,7 @@ -/* config.h. Generated manually for windows / vc++ 6.0 */ +/* + * @(#) $Id$ + * config.h. Generated manually for windows / vc++ 6.0 + */ /* Define if the SMIv1/v2 parser shall be active. */ #define BACKEND_SMI

1 0

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

2007

2006

2005

2004

2003

2002

2001

2000

1999

Libsmi April 2008