🚀 go-pugleaf

I need help on restoring printer settings from a registry key. The
backup code works fine, retrieves printer settings and stores them
into the registry. Here is the restore code:

fPrtCfg is the printer configuration data buffer
fPrtCfgSize is it's byte size

/*********************************************************************
***********/
FIRST VERSION:

TPrinter * prt=Printer();

char * Device[256];
char * Driver[256];
char * Port [256];

prt->GetPrinter(Device,Driver,Port,DevMode); // Get printer device
name etc.


HGLOBAL myHandle=GlobalAlloc(GHND,fPrtCfgSize); // allocate a global
handle
void * pDevMode = (void *)GlobalLock(myHandle); // get the memory
pointer

memcpy(pDevMode,fPrtCfg,fPrtCfgSize); // fill it with previously
stored data
GlobalUnlock( (void*)DevMode ); // release memory (keep the handle)

prt->SetPrinter(Device,Driver,Port,(THandle)myHandle); //  GlobalFree
will be done by TPrinter

/*********************************************************************
***********/
SECOND VERSION:

 TPrinter * prt=Printer();

char * Device[256];
char * Driver[256];
char * Port [256];

prt->GetPrinter(Device,Driver,Port,DevMode); // Get printer device
name etc.


HANDLE prtHandle;
if (OpenPrinter(Device,&prtHandle,NULL)!=0) // retrieve printer handle
needed by document properties
    {
    unsigned size=DocumentProperties(0,prtHandle,Device,NULL,NULL, 0);
// retrieve size needed

    HGLOBAL myHandle=GlobalAlloc(GHND,size); // allocate a global
handle of needed size
    void * pDevMode = (void *)GlobalLock(myHandle);  // get the memory
pointer

    memcpy(pDevMode,fPrtCfg,std::min(size,fPrtCfgSize)); // fill it
with previously stored data, use std::min to prevent buffer overflow


DocumentProperties(0,prtHandle,Device,(DEVMODE*)pDevMode,(DEVMODE*)pDe
vMode, DM_IN_BUFFER|DM_OUT_BUFFER);
    // do I need to set pDevMode->dmFields? I don't know what fields
changed, potentially all...

    GlobalUnlock( (void*)DevMode );

    ClosePrinter(prtHandle);
    prt->SetPrinter(Device,Driver,Port,(THandle)myHandle); //
GlobalFree will be done by TPrinter
    }

/*********************************************************************
***********/

Both work.
The second performs the DocumentProperties validation required by
Windows but I don't know if I use it fine.
To execute it I need the printer handle that i can retrieve only with
an OpenPrinter followed by a ClosePrinter. Does it conflict with
TPrinter encapsulation of the printer object?
I think the second version is better.
Any idea and help appreciated.
Thanks