Velocity Reviews - Computer Hardware Reviews

Velocity Reviews > Newsgroups > Programming > ASP .Net > Memory Allication (Managed VC++ DLL to Unmanaged DLL) [Second Try]

Reply
Thread Tools

Memory Allication (Managed VC++ DLL to Unmanaged DLL) [Second Try]

 
 
Weston Fryatt
Guest
Posts: n/a
 
      12-29-2004
(Sorry for spamming multiple groups, But I need a solution to this problem)

I think this should be a simple question on Memory Allocation in a managed DLL and passing a memory pointer over to an unmanaged DLL.

I have a "Unmanaged" Client DLL that I'm creating a Managed "wrapper" to be used in VB.Net and/or C#..

In the my Client DLL (unmanaged), There are several functions where I need to allocate a "client side" memory buffer to marshall data into from the server side of the application.

Normally in VC++ 7.1 (unmanaged) I would normally just malloc the memory that I needed:

UBYTE *pDataPtr = NULL;
DWORD dwDataSize = 0;

try
{
GetServerSideDataSize(dwDataSize);
pDataPtr = (UBYTE *)malloc(dwDataSize);
if (pDataPtr)
{
GetServerSideData(pDataPtr);
// Do something with pData Now..
free(pDataPtr);
pDataPtr = NULL;
}
}
catch(...)
{
if (pDataPtr)
free(pDataPtr);
}

In my Managed DLL wrapper I've done this so far which fails when I call "new" on the VB.Net side..

namespace CDMSClient_SQL
{
[DllImport("CDMSClient.dll", EntryPoint = "SQLGetRows", CharSet = Ansi)]
STATUS SQLGetRows(const Int32 RowCount, const Int32 DataSize, UBYTE *Data, String *Columns, String *From, String *Where, String *OrderBy);

[DllImport("CDMSClient.dll", EntryPoint = "SQLGetRowInfo", CharSet = Ansi)]
STATUS SQLGetRowInfo(Int32 &RowCount, Int32 &ColumnCount, Int32 &DataSize, String *Columns, String *From, String *Where);
}


Array *CCDMSClientInterface::SQLGetRows(const Int32 MaxRows, String *Columns, String *From, String *Where, String *OrderBy)
{
STATUS Status = EC_OK;
Int32 dwRowCount = 0;
Int32 dwColumnCount = 0;
Int32 dwDataSize = 0;
Array *Data = NULL;
UBYTE __pin *pDataPtr = NULL;

Status = CDMSClient_SQL::SQLGetRowInfo(dwRowCount, dwColumnCount, dwDataSize, Columns, From, Where);
if (Status == EC_OK)
{
pDataPtr = (UBYTE *)new char[dwDataSize]; // <--- This is the line that fails in the Managed VC++ Side
Status = CDMSClient_SQL::SQLGetRows(dwRowCount, dwDataSize, pDataPtr, "*", "CDMS_Config", "", "");
if (Status == EC_OK)
Data = Convert_CDBArray_2_Object(pDataPtr);
}

return (Data);
}

VB.Net Side of code:

Imports ClientInterface = CDMSClientDotNet.CCDMSClientInterface

Module Module1

Sub Main()
Dim ci As ClientInterface = New ClientInterface
Dim Data As Array
Dim Status As Int32

Status = ci.SetApplicationID("43C1F2F1-CB68-4B89-A8C8-5E0C42CE4866")
If Status = 1 Then
Status = ci.SetDomain("Primary")
If Status = 1 Then
Status = ci.CreateServerContext()
If Status = 1 Then
Status = ci.Login("admin", "optical")
If Status = 1 Then
Console.WriteLine("Logged In!")

Data = ci.SQLGetRows(1000, "*", "CDMS_Config", "", "") // <---This is the line that fails in the VB.Net side

ci.Logout()
Else
Console.WriteLine("Failed to Login!")
End If
ci.DeleteServerContext()
Else
Console.WriteLine("Failed to Create Server Context!")
End If
Else
Console.WriteLine("Failed to Set Domain!")
End If
Else
Console.WriteLine("Failed to Set ApplicationID!")
End If
End Sub

End Module


I have many functions that pass data from the server side to the client side via a memory buffer. This data in the memory buffer is parsed and put back into a dynamic array (Custom C++ class called a CDBArray). The first function that I'm trying to get to work, fetches a database table from the server side and marshalls it back over to the client side. The Server and Client is using RPC to marshall data back and forth. (Just a side note, The client side can not access the database directly via ODBC or anything else. It must go thought my client dll. This is done for security reason to keep the client side locked down.)

Any ideas or suggestion are more than welcomed!!

Thanks Again!
Weston Fryatt

 
Reply With Quote
 
 
 
 
Richard Grimes [MVP]
Guest
Posts: n/a
 
      12-30-2004
Weston Fryatt" <wfryatt "at wrote:
> In my Managed DLL wrapper I've done this so far which fails when I
> call "new" on the VB.Net side..


> namespace CDMSClient_SQL
> {
> [DllImport("CDMSClient.dll", EntryPoint = "SQLGetRows", CharSet =
> Ansi)]
> STATUS SQLGetRows(const Int32 RowCount, const Int32 DataSize,
> UBYTE *Data, String *Columns, String *From, String *Where, String
> *OrderBy);
>
> [DllImport("CDMSClient.dll", EntryPoint = "SQLGetRowInfo",
> CharSet = Ansi)]
> STATUS SQLGetRowInfo(Int32 &RowCount, Int32 &ColumnCount,
> Int32 &DataSize, String *Columns, String *From, String *Where);
> }


This is managed C++? If so, why are you using platform invoke to access the
methods. Why don't you use an import library (.lib) and use IJW?

> Array *CCDMSClientInterface::SQLGetRows(const Int32 MaxRows, String
> *Columns, String *From, String *Where, String *OrderBy)


Note that const means nothing to .NET and since you are calling this class
with VB.NET the constness of MaxRows is ignored. What is Array? Is this
System::Array? If so, why are you returning an untyped array?

> pDataPtr = (UBYTE *)new char[dwDataSize]; // <--- This is
> the line that fails in the Managed VC++ Side


Since you are using unmanaged pointers this should call the unmanaged new.
To make absolutely sure you can write it as

pDataPtr = (UBYTE *)__nogc new char[dwDataSize];

are you sure that this is where the error lies? What is the error message?
have you tried testing dwDataSize to see if it is not zero, and pDataPtr to
see if it is not null?

> Status = CDMSClient_SQL::SQLGetRows(dwRowCount, dwDataSize,
> pDataPtr, "*", "CDMS_Config", "", "");
> if (Status == EC_OK)
> Data = Convert_CDBArray_2_Object(pDataPtr);
> }


Where do you call delete []?

If SQLGetRows does not care where the memory comes from you can use
Marshal::AllocHGlobal or Marshal::AllocCoTaskMem and then use the
appropriate FreeCoTaskMem and FreeHGlobal. Both of these returns a IntPtr
which can be cast to a void* and then assigned to a pinning pointer.
However, the C++ new should work fine as long as you call delete []
somewhere. (Personally I would bracket the code in try/__finally and call
delete [] in the __finally clause.)

>
> Imports ClientInterface = CDMSClientDotNet.CCDMSClientInterface


This is poor naming, it's a class, not an interface, you cannot call New on
an interface!

> Module Module1
>
> Dim Data As Array
>
> Data = ci.SQLGetRows(1000, "*",
> "CDMS_Config", "", "") // <---This is the line that fails in the VB.Net
> side


Again System.Array is not much use. You should write the managed C++ to
return a typed array instead.

> I have many functions that pass data from the server side to the
> client side via a memory buffer. This data in the memory buffer is
> parsed and put back into a dynamic array (Custom C++ class called a
> CDBArray). The first function that I'm trying to get to work,
> fetches a database table from the server side and marshalls it back
> over to the client side. The Server and Client is using RPC to
> marshall data back and forth. (Just a side note, The client side can


RPC should handle any type of memory you pass to it, the RPC layer will
allocate the appropriate buffer. The only time that you'll get a problem is
if you're returning a new array from the RPC method. Are any of the pointers
in SQLGetRows [in,out]?

Richard
--
www.richardgrimes.com
my email http://www.velocityreviews.com/forums/(E-Mail Removed) is encrypted with ROT13 (www.rot13.org)


 
Reply With Quote
 
 
 
 
Weston Fryatt
Guest
Posts: n/a
 
      01-01-2005
Thanks Richard!

Marshal::AllocHGlobal() did the trick!




"Richard Grimes [MVP]" <read my sig> wrote in message
news:(E-Mail Removed)...
> Weston Fryatt" <wfryatt "at wrote:
> > In my Managed DLL wrapper I've done this so far which fails when I
> > call "new" on the VB.Net side..

>
> > namespace CDMSClient_SQL
> > {
> > [DllImport("CDMSClient.dll", EntryPoint = "SQLGetRows", CharSet =
> > Ansi)]
> > STATUS SQLGetRows(const Int32 RowCount, const Int32 DataSize,
> > UBYTE *Data, String *Columns, String *From, String *Where, String
> > *OrderBy);
> >
> > [DllImport("CDMSClient.dll", EntryPoint = "SQLGetRowInfo",
> > CharSet = Ansi)]
> > STATUS SQLGetRowInfo(Int32 &RowCount, Int32 &ColumnCount,
> > Int32 &DataSize, String *Columns, String *From, String *Where);
> > }

>
> This is managed C++? If so, why are you using platform invoke to access

the
> methods. Why don't you use an import library (.lib) and use IJW?
>
> > Array *CCDMSClientInterface::SQLGetRows(const Int32 MaxRows, String
> > *Columns, String *From, String *Where, String *OrderBy)

>
> Note that const means nothing to .NET and since you are calling this class
> with VB.NET the constness of MaxRows is ignored. What is Array? Is this
> System::Array? If so, why are you returning an untyped array?
>
> > pDataPtr = (UBYTE *)new char[dwDataSize]; // <--- This is
> > the line that fails in the Managed VC++ Side

>
> Since you are using unmanaged pointers this should call the unmanaged new.
> To make absolutely sure you can write it as
>
> pDataPtr = (UBYTE *)__nogc new char[dwDataSize];
>
> are you sure that this is where the error lies? What is the error message?
> have you tried testing dwDataSize to see if it is not zero, and pDataPtr

to
> see if it is not null?
>
> > Status = CDMSClient_SQL::SQLGetRows(dwRowCount, dwDataSize,
> > pDataPtr, "*", "CDMS_Config", "", "");
> > if (Status == EC_OK)
> > Data = Convert_CDBArray_2_Object(pDataPtr);
> > }

>
> Where do you call delete []?
>
> If SQLGetRows does not care where the memory comes from you can use
> Marshal::AllocHGlobal or Marshal::AllocCoTaskMem and then use the
> appropriate FreeCoTaskMem and FreeHGlobal. Both of these returns a IntPtr
> which can be cast to a void* and then assigned to a pinning pointer.
> However, the C++ new should work fine as long as you call delete []
> somewhere. (Personally I would bracket the code in try/__finally and call
> delete [] in the __finally clause.)
>
> >
> > Imports ClientInterface = CDMSClientDotNet.CCDMSClientInterface

>
> This is poor naming, it's a class, not an interface, you cannot call New

on
> an interface!
>
> > Module Module1
> >
> > Dim Data As Array
> >
> > Data = ci.SQLGetRows(1000, "*",
> > "CDMS_Config", "", "") // <---This is the line that fails in the VB.Net
> > side

>
> Again System.Array is not much use. You should write the managed C++ to
> return a typed array instead.
>
> > I have many functions that pass data from the server side to the
> > client side via a memory buffer. This data in the memory buffer is
> > parsed and put back into a dynamic array (Custom C++ class called a
> > CDBArray). The first function that I'm trying to get to work,
> > fetches a database table from the server side and marshalls it back
> > over to the client side. The Server and Client is using RPC to
> > marshall data back and forth. (Just a side note, The client side can

>
> RPC should handle any type of memory you pass to it, the RPC layer will
> allocate the appropriate buffer. The only time that you'll get a problem

is
> if you're returning a new array from the RPC method. Are any of the

pointers
> in SQLGetRows [in,out]?
>
> Richard
> --
> www.richardgrimes.com
> my email (E-Mail Removed) is encrypted with ROT13 (www.rot13.org)
>
>



 
Reply With Quote
 
 
 
Reply

Thread Tools

Posting Rules
You may not post new threads
You may not post replies
You may not post attachments
You may not edit your posts

BB code is On
Smilies are On
[IMG] code is On
HTML code is Off
Trackbacks are On
Pingbacks are On
Refbacks are Off


Similar Threads
Thread Thread Starter Forum Replies Last Post
Memory problem in asp.net calling unmanaged DLL monika.saxena@gmail.com ASP .Net 0 06-13-2005 03:18 PM
msvcrt.dll, msvcirt.dll, msvcrt20.dll and msvcrt40.dll, explanation please! Snoopy NZ Computing 16 08-25-2003 12:34 PM
unmanaged dll DllNotFoundException Petr Makagon ASP .Net 1 08-12-2003 01:40 AM
Problems using a unmanaged DLL from ASP.NET Luiz Miranda ASP .Net 0 07-30-2003 07:19 PM
How to access unmanaged DLL from within ASP.NET (using VB.NET) Sacha Korell ASP .Net 0 07-25-2003 08:55 PM



Advertisments