Skip to content

Use NativeMemory in System.Data.Odbc #85966

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
xtqqczze wants to merge 4 commits into from
Closed

Use NativeMemory in System.Data.Odbc #85966

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
a99952d
Use `NativeMemory` in `System.Data.Odbc`
xtqqczze May 9, 2023
68c1005
Workaround lack of `NativeMemory.Clear` on .NET 6
xtqqczze May 9, 2023
b593c9b
Use `Unsafe.InitBlock`
xtqqczze May 9, 2023
08e2ed0
Merge branch 'main' into NativeMemory.Odbc
xtqqczze Sep 11, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Next Next commit
Use NativeMemory in System.Data.Odbc
  • Loading branch information
@xtqqczze
xtqqczze committed May 9, 2023
commit a99952d7e73e1028e4c38489fa221e2106fd5ea7
28 changes: 0 additions & 28 deletions src/libraries/System.Data.Odbc/src/Common/System/Data/Common/SafeNativeMethods.cs
View file
Open in desktop

This file was deleted.

24 changes: 6 additions & 18 deletions src/libraries/System.Data.Odbc/src/Common/System/Data/ProviderBase/DbBuffer.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -14,21 +14,12 @@ internal abstract class DbBuffer : SafeHandle
{
private readonly int _bufferLength;

protected DbBuffer(int initialSize) : base(IntPtr.Zero, true)
protected unsafe DbBuffer(int initialSize) : base(IntPtr.Zero, true)
{
if (0 < initialSize)
{
_bufferLength = initialSize;

try { }
finally
{
base.handle = SafeNativeMethods.LocalAlloc((IntPtr)initialSize);
}
if (IntPtr.Zero == base.handle)
{
throw new OutOfMemoryException();
}
base.handle = (IntPtr)NativeMemory.AllocZeroed((uint)initialSize);
}
}

Expand Down Expand Up @@ -368,15 +359,12 @@ internal unsafe float ReadSingle(int offset)
return BitConverter.Int32BitsToSingle(value);
}

protected override bool ReleaseHandle()
protected override unsafe bool ReleaseHandle()
{
// NOTE: The SafeHandle class guarantees this will be called exactly once.
IntPtr ptr = base.handle;
base.handle = IntPtr.Zero;
if (IntPtr.Zero != ptr)
{
SafeNativeMethods.LocalFree(ptr);
}
NativeMemory.Free((void*)ptr);
return true;
}

Expand Down Expand Up @@ -639,7 +627,7 @@ internal unsafe void WriteSingle(int offset, float value)
WriteInt32(offset, BitConverter.SingleToInt32Bits(value));
}

internal void ZeroMemory()
internal unsafe void ZeroMemory()
{
bool mustRelease = false;

Expand All @@ -648,7 +636,7 @@ internal void ZeroMemory()
DangerousAddRef(ref mustRelease);

IntPtr ptr = DangerousGetHandle();
SafeNativeMethods.ZeroMemory(ptr, Length);
NativeMemory.Clear((void*)ptr, (uint)Length);
}
finally
{
Expand Down
1 change: 0 additions & 1 deletion src/libraries/System.Data.Odbc/src/System.Data.Odbc.csproj
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -42,7 +42,6 @@ System.Data.Odbc.OdbcTransaction</PackageDescription>
<Compile Include="$(CommonPath)System\Data\Common\NameValuePair.cs"
Link="Common\System\Data\Common\NameValuePair.cs" />
<Compile Include="Common\System\Data\Common\NameValuePermission.cs" />
<Compile Include="Common\System\Data\Common\SafeNativeMethods.cs" />
<Compile Include="Common\System\Data\DataStorage.cs" />
<Compile Include="Common\System\Data\ProviderBase\DbBuffer.cs" />
<Compile Include="$(CommonPath)System\Data\Common\FieldNameLookup.cs"
Expand Down