Problem mit realloc?

[heinz (Gast)]

Hallo,
ich versuch mich gerade an einem Slicer für STL Modelle und hab ein 
Problem
mit der Speicherallocation

OS ist Windows XP
Compiler PellesC
Sprache C

Bis auf den Fehler alles rausgelöscht

aufgerufen wird aus WindowProc durch Tastendruck

typedef struct{
  float xn,yn,zn;
  float x1,y1,z1;
  float x2,y2,z2;
  float x3,y3,z3;
} TRIANGLE;



int ReadSTL(void){
int i,max;
TRIANGLE *t=NULL;

  max=2000;
  t=malloc(max*sizeof(TRIANGLE));

  max=30000;
  realloc(t,max*sizeof(TRIANGLE));
  if (t==NULL) return -1;
  for (i=0;i<=2000;i++){
    t[i].z3=(FLOAT)5.5;
  }
  return 0;
}

gibt beim schreiben auf t[i].z3 (i=12) eine Zugriffsverletzung
Wenn ich den realloc rausnehm funktioniert es.
Was sehe ich da nicht?

Gruss heinz

[Peter II (Gast)]

heinz schrieb:
> Wenn ich den realloc rausnehm funktioniert es.
> Was sehe ich da nicht?

schau doch mal in der doku nach wie realloc funktioniert, achte dabei 
mal auf das "return" der funktion!

[heinz (Gast)]

laut Doku sollte der realloc einen Nullpointer zurückgeben wenn er fehl 
schlägt.

Ich kann mit dem Debugger verfolgen, dass der in dem Programmteil bleibt 
bis zum bitteren Ende

Das wollte ich eigentlich in "PC Hard & Software" posten, kann das 
jemand verschieben?

[Stefan E. (sternst)]

heinz schrieb:
> laut Doku sollte der realloc einen Nullpointer zurückgeben wenn er fehl
> schlägt.

Und du wertest die Rückgabe überhaupt nicht aus.

[Peter II (Gast)]

heinz schrieb:
> laut Doku sollte der realloc einen Nullpointer zurückgeben wenn er fehl
> schlägt.

und was was liefert er zurück wenn es keinen fehler gibt?

tipp: wie soll sich denn t ändert, wenn du ihn an eine funktion 
übergibst?

[Bernhard M. (boregard)]

Hallo,

realloc kann den übergebenen Wert von t nicht ändern, deshalb gibt es 
den neuen Wert zurück.
Also so anwenden:

typedef struct{
  float xn,yn,zn;
  float x1,y1,z1;
  float x2,y2,z2;
  float x3,y3,z3;
} TRIANGLE;



int ReadSTL(void){
int i,max;
TRIANGLE *t=NULL;

  max=2000;
  t=malloc(max*sizeof(TRIANGLE));

  max=30000;
  t = realloc(t,max*sizeof(TRIANGLE));
  if (t==NULL) return -1;
  for (i=0;i<=2000;i++){
    t[i].z3=(FLOAT)5.5;
  }
  return 0;
}

[Karl H. (kbuchegg)]

heinz schrieb:
> laut Doku sollte der realloc einen Nullpointer zurückgeben wenn er fehl
> schlägt.

Und was liefert er, wenn er nicht fehl schlägt?

[heinz (Gast)]

@Bernhard M
Autsch - manchmal bin blind
Danke, auch an die anderen

heinz

[Peter II (Gast)]

Bernhard M. schrieb:
> Bernhard M.
>         (boregard)

schechtest Beispiel. Denn wenn realloc NULL zurückliefert, kannst du 
deinen alten speicher nicht mehr freigeben!

http://msdn.microsoft.com/en-us/library/xbebcx7d.aspx

[heinz (Gast)]

Das wahr schon klar

t1 = realloc(t,max*sizeof(TRIANGLE));
if (t1!=NULL)
  t=t1
else
...

Ich hab nur nicht gemerkt dass ich überhaupt kein lvalue hab.

[Andreas S. (Firma: Schweigstill IT) (schweigstill)]

Es gibt noch ein weiteres potentielles Problem, nämlich das Alignment 
eines Arrays von TRIANGLE. Daher kann das Array mehr Speicher benötigen 
als max*sizeof(TRIANGLE).

Die derzeitige Definitoin von TRIANGLE dürfte das Problem zwar nicht 
aufweisen, wohl jedoch:

typedef struct{
  float xn,yn,zn;
  float x1,y1,z1;
  float x2,y2,z2;
  float x3,y3,z3;
  uint8_t a;
} TRIANGLE;

[Peter II (Gast)]

Andreas Schweigstill schrieb:
> Es gibt noch ein weiteres potentielles Problem, nämlich das Alignment
> eines Arrays von TRIANGLE. Daher kann das Array mehr Speicher benötigen
> als max*sizeof(TRIANGLE).

nein das dürfte NIE ein problem werden. Dann dann könnte man überhaupt 
keine Programme schreiben.

ein index zugriff ist immer

p[i] = p + (i * sizeof(p))

entweder wird durch das Alignment das sizeof mit größer oder halt nicht. 
Dann gibt es auch keine lücken.

[Hans (Gast)]

Peter II schrieb:
> p[i] = p + (i * sizeof(p))

p[i] = *(p + (i * sizeof(p)))

dereferenzieren nicht vergessen :-)

[dadada (Gast)]

Hans schrieb:
> p[i] = *(p + (i * sizeof(p)))
>
> dereferenzieren nicht vergessen :-)

Das ist doch nicht richtig. Richtig ist

p[i]=*(p+i)

das 'sizeof' passiert automatisch.