Fixed bug 7217.

--HG-- branch : 6.1
lextudio · May 21, 2011 · 22e792e · 22e792e
1 parent 2923ba9
commit 22e792e
Show file tree

Hide file tree

Showing 7 changed files with 17 additions and 65 deletions.
diff --git a/Install/Install.wixproj b/Install/Install.wixproj
diff --git a/Install/Product.wxs b/Install/Product.wxs
diff --git a/SharpSnmpLib/AssemblyInfo.cs b/SharpSnmpLib/AssemblyInfo.cs
@@ -47,9 +47,9 @@
 //
 // You can specify all the values or you can use the default the Revision and
 // Build Numbers by using the '*' as shown below:
-[assembly: AssemblyVersion("6.1.011114.16")]
+[assembly: AssemblyVersion("6.1.010521.00")]
 #if (!CF)
-[assembly: AssemblyFileVersion("6.1.011114.16")]
+[assembly: AssemblyFileVersion("6.1.010521.00")]
 #endif
 [assembly: NeutralResourcesLanguage("en-US")]
 [assembly: CLSCompliant(true)]

diff --git a/SharpSnmpLib/ByteTool.cs b/SharpSnmpLib/ByteTool.cs
@@ -350,7 +350,7 @@ internal static byte[] GetRawBytes(IEnumerable<byte> orig, bool negative)
             // if sign bit is not correct, add an extra byte
             if ((list[list.Count - 1] & 0x80) != sign)
             {
-                list.Add(sign);
+                list.Add(flag);
             }
 
             list.Reverse();

diff --git a/SharpSnmpLib/Properties/AssemblyInfo.cs b/SharpSnmpLib/Properties/AssemblyInfo.cs
@@ -47,9 +47,9 @@
 //
 // You can specify all the values or you can use the default the Revision and
 // Build Numbers by using the '*' as shown below:
-[assembly: AssemblyVersion("6.1.011114.16")]
+[assembly: AssemblyVersion("6.1.010521.00")]
 #if (!CF)
-[assembly: AssemblyFileVersion("6.1.011114.16")]
+[assembly: AssemblyFileVersion("6.1.010521.00")]
 #endif
 [assembly: NeutralResourcesLanguage("en-US")]
 [assembly: CLSCompliant(true)]

diff --git a/SharpSnmpLib/Properties/AssemblyInfo.cs.bak b/SharpSnmpLib/Properties/AssemblyInfo.cs.bak
@@ -47,9 +47,9 @@ using System.Runtime.InteropServices;
 //
 // You can specify all the values or you can use the default the Revision and
 // Build Numbers by using the '*' as shown below:
-[assembly: AssemblyVersion("6.1.011114.16")]
+[assembly: AssemblyVersion("6.1.010521.00")]
 #if (!CF)
-[assembly: AssemblyFileVersion("6.1.011114.16")]
+[assembly: AssemblyFileVersion("6.1.010521.00")]
 #endif
 [assembly: NeutralResourcesLanguage("en-US")]
 [assembly: CLSCompliant(true)]

diff --git a/Tests/Tests/TestByteTool.cs b/Tests/Tests/TestByteTool.cs
@@ -64,6 +64,16 @@ public void TestWriteLongLength()
             ByteTool.WritePayloadLength(m, length);
             Assert.AreEqual(expected, m.ToArray());
         }
+
+        [Test]
+        public void TestNegative2()
+        {
+            // bug 7217 http://sharpsnmplib.codeplex.com/workitem/7217
+            Integer32 i = new Integer32(-250);
+            var result = DataFactory.CreateSnmpData(i.ToBytes());
+            Assert.AreEqual(SnmpType.Integer32, result.TypeCode);
+            Assert.AreEqual(-250, ((Integer32)result).ToInt32());
+        }
     }
 }
 #pragma warning restore 1591, 0618