🚀 go-pugleaf

"melk" <NOSPAMi_melk@libero.it> wrote in message
news:42443927@newsgroups.borland.com...

> Hi Thanks for your replay, I tried to use but it seems doesn't work!!!

Just saying it doesn't work says nothing about the actual problem you are
having.

> the first time I tried to use it, it gave me a memory write
> error so I only add a line to malloc "data" before memcpy...

That was not the correct thing to do.

> I olso tried to write to file the "data" buffer whitout pass from the
> "SafeArrayAccessData(v.parray, &data)" function and it write
> the picture file well!!!

You allocated memory for the buffer yourself instead of letting
SafeArrayAccessData() do it for you.

>         Graphics::TBitmap *bmp = new Graphics::TBitmap();
>         TJPEGImage *jpg = new TJPEGImage();

Wrap those in try...catch or try..finally blocks, or use an std::auto_ptr,
to ensure the memory is freed correctly in case an exception occurs.

>             SAFEARRAYBOUND bounds = {0, MemStream->Size-1};

Change that to this instead:

    SAFEARRAYBOUND bounds;
    bounds.lLbound = 0;
    bounds.cElements = MemStream->Size;

>             data = malloc(sizeof(byte)*MemStream->Size);

Get rid of that.

>             Response->End();

You don't need to call that since Flush() is called later.

>             free(data);

Get rid of that.


Gambit