Error en tiempo de ejecución al hacer un stream con fwrite

Iniciado por ccjrocks, 15 Junio 2013, 15:40 PM

0 Miembros y 2 Visitantes están viendo este tema.

ccjrocks

Muy buenas cracks, tengo un problemilla que me está frustrando bastante, os dejo el code para que veáis que falla, seguro que es una chorrada:


long tellFileSizeSTREAM(ifstream& FileToAnalize){
long startpoint=FileToAnalize.tellg();
long retorno=0;
FileToAnalize.seekg(0,ios_base::end);
retorno=FileToAnalize.tellg();
FileToAnalize.seekg(0,startpoint);
return retorno;
}


//A PARTIR DE AQUÍ ESTE CODE ESTÁ DENTRO DEL MAIN
ifstream readfile;
readfile.open(readpath.c_str(),ios::binary); //readpath es un string con la ruta
long readfilesize=tellFileSizeSTREAM(readfile);
while(readfile.good()){
char writebuffer;
readfile.read(&writebuffer,1);
outputfile.write(&writebuffer,1);
}


Lo extraño esque me da error en tiempo de ejecucción de violación de acceso a memoria, he probado a cambiar el código de mil maneras posibles pero no tengo ni idea de qué puede ser :-( A alguno os ha pasado esto?

Ah, y tengo que ir char a char porque quiero hacer una manipulación en ellos. A ver si alguno sabe lo que es. Gracas de antemano :silbar:

amchacon

Código (cpp) [Seleccionar]
while(readfile.good())

Esto no es correcto, para leer hasta final de archivo:

Código (cpp) [Seleccionar]
while(!readfile.eof())

Y aunque hagas una manipulación, no tienes que leer char a char. Puedes leer una línea (readfile.getline), modificarla y después escribirla.
Por favor, no me manden MP con dudas. Usen el foro, gracias.

¡Visita mi programa estrella!

Rar File Missing: Esteganografía en un Rar

ccjrocks

Me temo que ya lo había probado  :( (Lo de readfile.eof)


lapras

Pues no se lo único que veo es que en la función "tellFileSizeSTREAM" creo que deberías cambiar esta línea:
Código (cpp) [Seleccionar]
retorno=FileToAnalize.tellg()-startpoint;

Pero no estoy seguro por que no se para que usas esa función.

Una pregunta, ¿el archivo que intentas copiar tiene muchos espacios al principio?

ccjrocks

La función está bien, lo que hace es devolver el tamaño del fichero sin resetear el puntero del stream.

El principio del fichero es el siguiente:

MZ       ÿÿ  ¸       @                                   €   º ´ Í!¸LÍ!This program cannot be run in DOS mode.

$       PE  L š#²Q        à     >   4      ¾]       `    @                       à          @...                           d]  W    €  à.                   À      `                                                               H           .text   Ä=       >                    `.sdata  8   `      B              @  À.rsrc   à.   €   0   D              @  @.reloc      À      t              @  B                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                         ]      H     1  L,       P

0xDani

Supongo que estás haciendo un crypter o algo así... no?

Abres el archivo pasándole el flag ios::binary, pero no el flag ios::in. No sé si eso puede causar el error, pero prueba a cambiarlo.

Saludos.
I keep searching for something that I never seem to find, but maybe I won't, because I left it all behind!

I code for $$$
Hago trabajos en C/C++
Contactar por PM

lapras

#6
Vale, ya sé cual es un fallo. Resulta que read y write no se desplazan por el fichero solo leen y escriben empezando por el principio(a no ser que especifiques un offset).
Tendrías que usar put y get:
Código (cpp) [Seleccionar]
while(readfile.good()){
char writebuffer=readfile.get();

if(readfile.good())outputfile.put(writebuffer);
}
readfile.close();
outputfile.close();


A ver si te funciona esto...

Por otra parte tengo que decir que el error que te da no tiene pinta de ser por eso. Tengo la impresión de que en algún lugar se ha modificado un puntero. La dirección 0x20202020 es muy sospechosa, 0x20 en ascii es un espacio. ¿Cabe la posibilidad de que el error esté en otro sitio?

ccjrocks

#7
Cita de: 0xDani en 15 Junio 2013, 19:22 PM
Supongo que estás haciendo un crypter o algo así... no?

Abres el archivo pasándole el flag ios::binary, pero no el flag ios::in. No sé si eso puede causar el error, pero prueba a cambiarlo.

Saludos.

Creo que no le he pasado el flag ios::in no?? :-\
Y sí, es un crypter  ;D

Cita de: lapras en 15 Junio 2013, 20:13 PM
Vale, ya sé cual es un fallo. Resulta que read y write no se desplazan por el fichero solo leen y escriben empezando por el principio(a no ser que especifiques un offset).
Tendrías que usar put y get:
Código (cpp) [Seleccionar]
while(readfile.good()){
char writebuffer=readfile.get();

if(readfile.good())outputfile.put(writebuffer);
}
readfile.close();
outputfile.close();


A ver si te funciona esto...

Por otra parte tengo que decir que el error que te da no tiene pinta de ser por eso. Tengo la impresión de que en algún lugar se ha modificado un puntero. La dirección 0x20202020 es muy sospechosa, 0x20 en ascii es un espacio. ¿Cabe la posibilidad de que el error esté en otro sitio?

He probado tu code y esta vez me da el error 0x05, que cambiando el code de otras formas también me daba. Los errores que recibo probando diferentes codes son el 0x05 y 0x20 siempre.

Y he estado investigando y te diré que el read SÍ desplaza el fichero, osea que seguimos como al principio...

Creo que debe ser algo relacionado con el final de fichero, porque he probado también:

while(!readfile.bad()){


char writebuffer;
readfile.read(&writebuffer,1);
outputfile.write(&writebuffer,1);

}


Y funciona, lo que pasa que no para, se vuelve loco y escribe el fichero una y otra vez hasta que lo cierras. Si pruebo con:

while(!readfile.eof()){


char writebuffer;
readfile.read(&writebuffer,1);
outputfile.write(&writebuffer,1);

}


Me lee sólo parte del fichero, es decir, el fichero origen ocupa 28K y el de destino 30K, osea que para antes de llegar al final.

Prácticamente ahora sólo queda comprobar el final de fichero, se os ocurre algún método¿? (ya que con eof()) no lo conseguimos)

EDITO: Con este código pasa lo mismo que con el último ejemplo:
while(!readfile.bad()){

long actpos=readfile.tellg();
if(actpos==readfilesize){
break;
}
char writebuffer;
readfile.read(&writebuffer,1);
outputfile.write(&writebuffer,1);

}

0xDani

No has pensado en hacer algo como esto?:

Código (cpp) [Seleccionar]

uint64_t getFileSize(istream& stream)
{
uint64_t cur, ret;
cur = stream.tellg();
stream.seekg(0, ios::end);
ret = stream.tellg();
stream.seekg(cur, ios::beg);

return ret;
}

ifstream readfile(readpath.c_str(), ios::binary|ios::in);

uint64_t fileSize = getFileSize(readfile);
char *buf = new char[fileSize];
readfile.read(buf, fileSize);

for(size_t i=0; i<fileSize; i++)
{
/* Haces lo que quieras con buf[i] */
outputfile.put(buf[i]);
}

delete[] buf;


Lo veo un poco menos confuso no?
I keep searching for something that I never seem to find, but maybe I won't, because I left it all behind!

I code for $$$
Hago trabajos en C/C++
Contactar por PM

lapras

#9
Pues es raro que no te funcione mi código porque a mi si me funciona, puede que el error esté en otro sitio.
Mi código completo es este:
Código (cpp) [Seleccionar]
#include<iostream>
#include<fstream>

using namespace std;

int main(){
ofstream outputfile;
outputfile.open("file1",ios::out);
ifstream readfile;
readfile.open("file2",ios::in);


while(readfile.good()){
char writebuffer=readfile.get();

if(readfile.good())outputfile.put(writebuffer);
}
readfile.close();
outputfile.close();
}


Tenías razón sobre lo del read y write, si que desplazan.
Otra manera que me funciona es esta:

Código (cpp) [Seleccionar]
#include<iostream>
#include<fstream>

using namespace std;

int main(){
ofstream outputfile;
outputfile.open("file1",ios::out);
ifstream readfile;
readfile.open("file2",ios::in);

char writebuffer;
while(readfile.read(&writebuffer,1)){
if(readfile.good())outputfile.write(&writebuffer,1);
}
readfile.close();
outputfile.close();
}