Merge pull request #935 from Amberg/TrySendFrameLenghtFix

Fix TrySendFrame if buffer is bigger than length

Author: somdoron

src/NetMQ.Tests/OutgoingSocketExtensionsTests.cs

@@ -1,4 +1,6 @@
 using System;
+using System.Linq;
+using System.Text;
 using Xunit;
 
 namespace NetMQ.Tests
@@ -322,5 +324,22 @@ public void TrySignalFailedTest()
 
             Assert.False(socket.TrySignalOK());
         }
+
+        [Fact]
+        public void TrySendFrameBiggerBufferThanLength()
+        {
+	        var buffer = new byte[64];
+	        var data = Encoding.ASCII.GetBytes("Hello there");
+	        data.CopyTo(buffer, 0);
+	        var socket = new MockOutgoingSocket((ref Msg msg, TimeSpan timeout, bool more) =>
+	        {
+		        Assert.Equal(TimeSpan.Zero, timeout);
+		        Assert.True(data.SequenceEqual(msg.ToArray()));
+		        Assert.False(more);
+		        return true;
+	        });
+
+	        Assert.True(socket.TrySendFrame(TimeSpan.Zero, buffer, data.Length));
+        }
     }
 }

src/NetMQ/OutgoingSocketExtensions.cs

@@ -104,7 +104,7 @@ public static bool TrySendFrame(this IOutgoingSocket socket, TimeSpan timeout, b
         {
             var msg = new Msg();
             msg.InitPool(length);
-            data.CopyTo(msg);
+            data.Slice(0, length).CopyTo(msg);
             if (!socket.TrySend(ref msg, timeout, more))
             {
                 msg.Close();