<html dir="ltr">
<head>
<meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1">
<style type="text/css" id="owaParaStyle"></style>
</head>
<body fpstyle="1" ocsi="0">
<div style="direction: ltr;font-family: Tahoma;color: #000000;font-size: 10pt;">Bill, thanks for the patch.  We will take a look at is as soon as we finish editing this backlog of packets.
<div><br>
</div>
<div>Russ</div>
<div><br>
<div style="font-family: Times New Roman; color: #000000; font-size: 16px">
<hr tabindex="-1">
<div id="divRpF142683" style="direction: ltr; "><font face="Tahoma" size="2" color="#000000"><b>From:</b> Bill Parker [wp02855@...2499...]<br>
<b>Sent:</b> Sunday, April 13, 2014 3:27 PM<br>
<b>To:</b> snort-devel@lists.sourceforge.net<br>
<b>Subject:</b> [Snort-devel] [SNORT-DEVEL] modification to switch()/case in snprintf.c for NetVI<br>
</font><br>
</div>
<div></div>
<div>
<div dir="ltr">Hello All,<br>
<br>
    In reviewing some code in NetVI, directory 'src', file 'snprintf.c'<br>
I found a rather lengthy switch()/case workup consisting of the following:<br>
<br>
                    case '1':<br>
                    case '2':<br>
                    case '3':<br>
                    case '4':<br>
                    case '5':<br>
                    case '6':<br>
                    case '7':<br>
                    case '8':<br>
                    case '9':<br>
                    <br>
Which could be written as:<br>
<br>
                    case '1' ... '9':<br>
                    <br>
since ASCII values for '1' thru '9' are in order in the ASCII<br>
table.  The patch file which modifies this is below:<br>
<br>
--- snprintf.c.orig     2014-04-10 04:39:57.132977816 -0700<br>
+++ snprintf.c  2014-04-10 04:40:45.194086136 -0700<br>
@@ -174,15 +174,7 @@<br>
                     case '-': ljust = 1; goto nextch;<br>
                     case '0': /* set zero padding if len not set */<br>
                         if(len==0 && !pointflag) zpad = '0';<br>
-                    case '1':<br>
-                    case '2':<br>
-                    case '3':<br>
-                    case '4':<br>
-                    case '5':<br>
-                    case '6':<br>
-                    case '7':<br>
-                    case '8':<br>
-                    case '9':<br>
+                    case '1' ... '9':<br>
                         if(pointflag)<br>
                             maxwidth = maxwidth*10 + ch - '0';<br>
                         else<br>
                         <br>
Would this be of any use in NetVI and/or Snort/DAQ?<br>
<br>
Bill Parker (wp02855 at gmail dot com)<br>
</div>
</div>
</div>
</div>
</div>
</body>
</html>