From: Sudha Bheemanna Date: Wed, 8 Feb 2017 05:57:13 +0000 (+0530) Subject: [BT C#] Fix crash in read manuf data X-Git-Tag: accepted/tizen/common/20170208.155048~1^2 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=919dc30b5cdb55269c5480f00135a1d38d33b429;p=platform%2Fcore%2Fcsapi%2Fbluetooth.git [BT C#] Fix crash in read manuf data Fixed crash in read manufacturer data. code clean-up Change-Id: I801395d6dade184881da3e3b8316d57a26227d7e Signed-off-by: Sudha Bheemanna --- diff --git a/Tizen.Network.Bluetooth/Tizen.Network.Bluetooth/BluetoothLeAdapterImpl.cs b/Tizen.Network.Bluetooth/Tizen.Network.Bluetooth/BluetoothLeAdapterImpl.cs index a6773af..1fb02de 100755 --- a/Tizen.Network.Bluetooth/Tizen.Network.Bluetooth/BluetoothLeAdapterImpl.cs +++ b/Tizen.Network.Bluetooth/Tizen.Network.Bluetooth/BluetoothLeAdapterImpl.cs @@ -359,8 +359,8 @@ namespace Tizen.Network.Bluetooth internal ManufacturerData GetScanResultManufacturerData(BluetoothLeScanData scanData, BluetoothLePacketType packetType) { - int dataId; - int dataLength; + int dataId = 0; + int dataLength = 0; IntPtr manufData; BluetoothLeScanDataStruct scanDataStruct = BluetoothUtils.ConvertLeScanDataToStruct (scanData); @@ -377,7 +377,10 @@ namespace Tizen.Network.Bluetooth data.Id = dataId; data.DataLength = dataLength; if (data.DataLength > 0) + { + data.Data = new byte[data.DataLength]; Marshal.Copy(manufData, data.Data, 0, data.DataLength); + } return data; } @@ -425,8 +428,6 @@ namespace Tizen.Network.Bluetooth BluetoothLeAdvertisingState state, IntPtr userData) => { Log.Info(Globals.LogTag, "Setting advertising state changed callback !! " ); - // TODO: check if this conversion is required - BluetoothLeAdvertiser leAdvertiser = (BluetoothLeAdvertiser)Marshal.PtrToStructure(advertiserHandle, typeof(BluetoothLeAdvertiser)); AdvertisingStateChangedEventArgs e = new AdvertisingStateChangedEventArgs(result, advertiserHandle, state); _advertisingStateChanged(null, e); };