[Tizen.Applications.Cion] Fix Cion (#3682)

- Fix handling returned data of ServerBase
- Fix parameter of CionClientSendPayloadAsync

Signed-off-by: Sangyoon Jang <jeremy.jang@samsung.com>

diff --git a/src/Tizen.Applications.Cion/Interop/Interop.Cion.cs b/src/Tizen.Applications.Cion/Interop/Interop.Cion.cs
index 76767300502f872a59560b744f593079807919f5..75d81ba0ce802283f7aa10bff8ebb077b21e3522 100644 (file)
--- a/src/Tizen.Applications.Cion/Interop/Interop.Cion.cs
+++ b/src/Tizen.Applications.Cion/Interop/Interop.Cion.cs
@@ -14,6 +14,9 @@
  * limitations under the License.
  */
 
+using System;
+using System.Runtime.InteropServices;
+
 internal static partial class Interop
 {
     internal static partial class Cion
@@ -31,5 +34,8 @@ internal static partial class Interop
             TimedOut = Tizen.Internals.Errors.ErrorCode.TimedOut,
             OperationFailed = -0x030C0000 | 0x01,
         }
+
+        [DllImport(Libraries.Libc, EntryPoint = "malloc")]
+        internal static extern IntPtr Malloc(int size);
     }
 }

diff --git a/src/Tizen.Applications.Cion/Interop/Interop.CionServer.cs b/src/Tizen.Applications.Cion/Interop/Interop.CionServer.cs
index 1b4824784dcc49f7d9c84dbd092cb988fd3ca3ef..c08d7d3de07313022742fdd81f0c6f5ea169f8fe 100644 (file)
--- a/src/Tizen.Applications.Cion/Interop/Interop.CionServer.cs
+++ b/src/Tizen.Applications.Cion/Interop/Interop.CionServer.cs
@@ -34,7 +34,7 @@ internal static partial class Interop
         internal delegate void CionServerConnectionResultCb(string serviceName, IntPtr peerInfo, IntPtr result, IntPtr userData);
 
         [UnmanagedFunctionPointer(CallingConvention.Cdecl)]
-        internal delegate void CionServerDataReceivedCb(string serviceName, IntPtr peerInfo, byte[] data, int dataSize, out byte[] returnData, out int returnDataSize, IntPtr userData);
+        internal delegate void CionServerDataReceivedCb(string serviceName, IntPtr peerInfo, byte[] data, int dataSize, out IntPtr returnData, out int returnDataSize, IntPtr userData);
 
         [UnmanagedFunctionPointer(CallingConvention.Cdecl)]
         internal delegate void CionServerPayloadReceivedCb(string serviceName, IntPtr peerInfo, IntPtr payload, int status, IntPtr userData);

diff --git a/src/Tizen.Applications.Cion/Interop/Interop.Libraries.cs b/src/Tizen.Applications.Cion/Interop/Interop.Libraries.cs
index 32af82835dc99fda671a55a94f074aa541e65c9d..0f6ed84588c84998fac399cd07e3a9a8e90cf96c 100644 (file)
--- a/src/Tizen.Applications.Cion/Interop/Interop.Libraries.cs
+++ b/src/Tizen.Applications.Cion/Interop/Interop.Libraries.cs
@@ -19,5 +19,6 @@ internal static partial class Interop
     internal static partial class Libraries
     {
         public const string Cion = "libcion.so.1";
+        public const string Libc = "libc.so.6";
     }
 }

diff --git a/src/Tizen.Applications.Cion/Tizen.Applications.Cion/ClientBase.cs b/src/Tizen.Applications.Cion/Tizen.Applications.Cion/ClientBase.cs
index 1b697279746692f5c1f8255bbec727ee258ecbac..aac5531325d22603bae2b57f455041099c16a1a5 100644 (file)
--- a/src/Tizen.Applications.Cion/Tizen.Applications.Cion/ClientBase.cs
+++ b/src/Tizen.Applications.Cion/Tizen.Applications.Cion/ClientBase.cs
@@ -36,7 +36,6 @@ namespace Tizen.Applications.Cion
         private Interop.CionClient.CionClientConnectionResultCb _connectionResultCb;
         private Interop.CionClient.CionClientPayloadReceivedCb _payloadRecievedCb;
         private Interop.CionClient.CionClientDisconnectedCb _disconnectedCb;
-        private Interop.CionClient.CionClientPayloadAsyncResultCb _payloadAsyncResultCb;
         private Dictionary<string, TaskCompletionSource<PayloadAsyncResult>> _tcsDictionary = new Dictionary<string, TaskCompletionSource<PayloadAsyncResult>>();
         private Dictionary<string, Interop.CionClient.CionClientPayloadAsyncResultCb> _payloadAsyncCbDictionary = new Dictionary<string, Interop.CionClient.CionClientPayloadAsyncResultCb>();
 
@@ -305,7 +304,7 @@ namespace Tizen.Applications.Cion
                     _tcsDictionary.Remove(resultPayload.PayloadId);
                 });
 
-            Interop.Cion.ErrorCode ret = Interop.CionClient.CionClientSendPayloadAsync(_handle, payload?._handle, _payloadAsyncResultCb, IntPtr.Zero);
+            Interop.Cion.ErrorCode ret = Interop.CionClient.CionClientSendPayloadAsync(_handle, payload?._handle, cb, IntPtr.Zero);
             if (ret != Interop.Cion.ErrorCode.None)
             {
                 throw CionErrorFactory.GetException(ret, "Failed to send payload.");

diff --git a/src/Tizen.Applications.Cion/Tizen.Applications.Cion/ServerBase.cs b/src/Tizen.Applications.Cion/Tizen.Applications.Cion/ServerBase.cs
index ddc59e664ff9ff07b657bb13f91321ea43a54847..f5ddd213bfc8a397d5b725c32cc71865eedb17a9 100644 (file)
--- a/src/Tizen.Applications.Cion/Tizen.Applications.Cion/ServerBase.cs
+++ b/src/Tizen.Applications.Cion/Tizen.Applications.Cion/ServerBase.cs
@@ -16,6 +16,7 @@
 
 using System;
 using System.Collections.Generic;
+using System.Runtime.InteropServices;
 using System.Threading.Tasks;
 
 namespace Tizen.Applications.Cion
@@ -121,17 +122,19 @@ namespace Tizen.Applications.Cion
             }
 
             _dataReceivedCb = new Interop.CionServer.CionServerDataReceivedCb(
-                (string service, IntPtr peerInfo, byte[] data, int dataSize, out byte[] returnData, out int returnDataSize, IntPtr userData) =>
+                (string service, IntPtr peerInfo, byte[] data, int dataSize, out IntPtr returnData, out int returnDataSize, IntPtr userData) =>
                 {
                     Interop.Cion.ErrorCode clone_ret = Interop.CionPeerInfo.CionPeerInfoClone(peerInfo, out PeerInfoSafeHandle clone);
                     if (clone_ret != Interop.Cion.ErrorCode.None)
                     {
                         Log.Error(LogTag, "Failed to clone peer info.");
-                        returnData = null;
+                        returnData = IntPtr.Zero;
                         returnDataSize = -1;
                     }
-                    returnData = OnDataReceived(data, new PeerInfo(clone));
-                    returnDataSize = returnData.Length;
+                    byte[] returnDataRaw = OnDataReceived(data, new PeerInfo(clone));
+                    returnDataSize = returnDataRaw.Length;
+                    returnData = Interop.Cion.Malloc(returnDataSize);
+                    Marshal.Copy(returnDataRaw, 0, returnData, returnDataSize);
                 });
             ret = Interop.CionServer.CionServerSetDataReceivedCb(_handle, _dataReceivedCb, IntPtr.Zero);
             if (ret != Interop.Cion.ErrorCode.None)