c progamm

[Smokey]

die funktion funktioniert wenn ich sie einmal aufrufe..

sobald sie aber zwei mal aufgerufen wird gibts einen speicherzugriffsfehler.

hat irgendwer eine ahnung woran das liegen könnte?

mfg smokey

PHP-Code:

int checkip(char *ip)
{
int count=0;
int ts1, ts2;
struct timeb ts;
FILE *logfile;
char *puffer;
char *iplog;
int ret=0;

logfile = fopen("log.dat","r");

ftime(&ts); 

while (fread(&puffer, 1, 1, logfile)>0)
{
    if(strcmp(&puffer,":"))
        strncat(iplog,&puffer,1);
        
    else
    {
        if(!strcmp(iplog,ip))
        {    
            fread(&puffer, 10, 1, logfile);
            break;
        }
        //printf("Gesperrt");
    strcpy(iplog,"");
    }
}

ts1 = convint(&puffer);
ftime(&ts); 
ts2 = ts.time;
ts1 = ts2-ts1;

if(ts1 < 1800)
ret=1;

fclose(logfile);
return ret;
} 


[helios]

wundert micht dass das überhaupt 1x geht. du verwendest lauter uninitialisierte pointer (iplog, puffer). einmal davon abgesehen dass du die pointer selber überschreibst.

[pong]

hat irgendwer eine ahnung woran das liegen könnte?

Schonmal was von Grundinitialisierung gehört..... kannst nur froh über deinen Compiler sein, der das zumindest vor dem Erstaufruf für dich macht

pong

[Biri]

ein lustiges programm + interessant, dass überhaupt was funktioniert.
sorry.

vor dem lesen von daten in die bereiche
char *puffer;
char *iplog;
würde es sich empfehlen, entsprechenden speicher zu alokieren (anzulegen).

char *irgendwas erzeugt einen pointer im char-format, also eine variable, welche die adresse eines speicherbereiches enthalten kann.
Genau diesen speicherbereich musst du zuvor jedoch anlegen.

statisch: char meinspeicher[50];
dynamisch mit "malloc" (syntax weiß ich jetzt net auswendig.

fg
-hannes

[Smokey]

so das probier ich gleich mal aus...

sorry kann mit pointern einfach ned.... es lebe die objekt orientierte sprache ^^

[Smokey]

habs geändert!

funktioniert jetzt kann also geschlossen werden

und nochmal danke an alle

[Biri]

hi,

schön, dass nun funktioniert!

btw. c++ ist z.B. eine objektorientierte sprache + hat pointer.
c# ebenfalls, jedoch nur in "unsafe" bereichen.

fg
-hannes