strcpy führt zum "Debug Assertion Failed"

MBCS-CITP

Offensichtlich ist jedoch

pOls->pOlsParams->pSearchCommand

NULL; die frage ist also wo kommt dieser Wert her?

Weiter: Der Gebrauch strcpy ist immer heikel. Man darf strcpy NUR verwenden, wenn man zeigen kann (am besten schriftlich in der Dokumentation bei nicht-trivalen Faellen), dass die Destination wirklich immer und in allen Faellen genug Speicher, hat, um die den Quellstring aufzunehmen; und wirklich nur dann.

reiser

Also der Speicher wird allokiert:

/* allocate memory for Ols data structure
-----------------------------------------*/

pOls = (Ols*) LAYER_calloc( 1U, sizeof(Ols), OLS_ALLOC_MODNAME, "Ols_Init" );

was dann in LAYER_malloc.h und LAYER_malloc.c auf folgendes rausläuft:
LAYER_malloc.h:

#define LAYER_calloc(num, size, categoryName, blockName)  internLAYER_calloc(num, size)

/*...*/

LAYER_malloc.c:

void* internLAYER_calloc (
	LAYER_Size	num,
	LAYER_Size	size
)
{
    /* disable lint error 421: (Warning -- Caution -- function 
       'calloc(unsigned int, unsigned int)' is considered dangerous 
       [MISRA Rule 118]) 
    */
    return calloc(num, size);      /*lint !e421 */
}

Ich denke mal das so die ganze struktur allokiert wird.
Ich hab diesen Teil des Codes nicht geschrieben.
Hab ich da jetzt nen Denkfehler drin?

Wird der Rückgabewert von LAYER_calloc() auf NULL überprüft?
Wie sieht die Struktur Ols denn aus? Kann es sein, dass pSearchCommand nur ein char* ist (was dem Namen nach logisch wäre) und kein internes Array? Dann muss man halt für pSearchCommand extra noch Speicher reservieren.

MBCS-CITP

pOls = (Ols*) LAYER_calloc( 1U, sizeof(Ols), OLS_ALLOC_MODNAME, "Ols_Init" );

Da liegt der Hund begraben. Die Struktur p0ls enthaelt offensichtlich nur einen Pointer auf pOlsParams und der wiederrum nur einen Pointer auf pSearchCommand (ansonsten waere der -> Zugriff nicht moeglich). Was Du also mit Deinem calloc machst, ist Speicher fuer den Pointer allociert, aber nicht den Speicherbereich auf die der Pinter zeigen muss. Das wird wohl bei dieser Funktion zweimal der Fall sein muessen fuer;

p0ls->pOlsParams

und fuer

pOls->pOlsParams->pSearchCommand

Da fehlen noch zwei calloc/mallocs.

reiser

OK, alles so gemacht!
Also zweimal speicher allociert und was in der Variablen gespeichert.
Und bekomme jetzt:

"Access Violation..."

tritt in strcat.asm auf.

MBCS-CITP

Wie allocierst Du denn den Steicher fuer das Array genau? Wie ist das Ergebnis bei strncpy (Rueckgabewert, Crash etc.)?

reiser

das ging mir jetzt mal richtig auf den sack und desshalb bin ich mit erfolg auf memcpy umgestiegen.

Aber, trotzdem danke für die hilfe. Hab einiges gelernt.

MfG

c.rackwitz

was fuer daten wolltest du mit strcpy kopieren?
waren die auch nullterminiert, wie es sich fuer strings gehoert?

reiser

Ich wollte einen String kopieren.
Und die waren auch nullterminiert, hab im Speicher explizit noch mal nachgekuckt.

c.rackwitz

...musst du wissen, ob das so stimmt...

war an der zieladresse genug freier speicher?

strcpy führt zum &quot;Debug Assertion Failed&quot;

strcpy führt zum "Debug Assertion Failed"