Discussion :

[guipe]

Bonjour, je m'explique.J'ai un programme que je dois corriger avec gdb.Il parait que l'erreur est facile à trouver mais je ne trouve pas.Si vous pouvez m'aider merci!

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
#include <sys/errno.h>
#include <string.h>
#include <stdlib.h>
#include <unistd.h>
#include "bio.h"
 
BFILE *bopen(const char *filename, const char *mode)
{
	BFILE *stream;
	int flags;
	int fd;
	char md;
 
	if (strcmp(mode,"r") == 0) {
		flags = O_RDONLY;
		md = BMODE_READ;
	} else {
		errno = EINVAL;
		return NULL;
	}
	if ((fd = open(filename, flags, PERM_FILE)) == -1)
		return NULL;
	if ((stream=(BFILE *)malloc(sizeof(BFILE)))!=NULL) {
		stream->fd=fd;
		stream->mode=md;
		stream->pos=0;
	}
	return stream;
}
 
ssize_t bread(void *buf, ssize_t size, BFILE *stream)
{
	char *ptr;
	ssize_t more;
 
	if ((stream==NULL)||(stream->mode!=BMODE_READ)) {
		errno = EBADF;
		return 0;
	}
	more=size;
	ptr=buf;
	while (stream->fill-stream->pos < more) {
		memcpy(ptr,&stream->buf[stream->pos],stream->fill-stream->pos);
		ptr+=stream->fill-stream->pos;
		stream->pos=0;
		if ((stream->fill=read(stream->fd,stream->buf,BBUFSIZ))<=0) {
			stream->eof=(stream->fill==0);
			stream->fill=0;
			return size-more;
		}
	}
	memcpy(ptr,&stream->buf[stream->pos],more);
	stream->pos+=more;
	return size;
}
 
int beof(BFILE *stream)
{
	return stream->eof;
}
 
int bclose(BFILE *stream)
{
	int r;
 
	if (stream==NULL) {
		errno = EBADF;
		return BEOF;
	}
	if ((r=close(stream->fd))==0)
		free(stream);
	return r;
}

Quand je fais run avec gdb, j'ai une erreur comme ceci en pareticulier:

if (stream==NULL) {
errno = EBADF;
*** stack smashing detected ***: /home/peano/Desktop/tp_systeme/bio/cat terminated

[Médinoc]

On n'a pas la définition de la structure BFILE.

[diogene]

On n'a pas non plus le code qui appelle ces fonctions (notamment bclose()) !

[guipe]

Voici le fichier bio.c

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
#include <sys/errno.h>
#include <string.h>
#include <stdlib.h>
#include <stdio.h>
#include <unistd.h>
#include "bio.h"

BFILE *bopen(const char *filename, const char *mode)
{
	BFILE *stream;
	int flags;
	int fd;
	char md;
	
	if (strcmp(mode,"r") == 0) {
		flags = O_RDONLY;
		md = BMODE_READ;
	} else {
		errno = EINVAL;
		return NULL;
	}
	if ((fd = open(filename, flags, PERM_FILE)) == -1)
		return NULL;
	if ((stream=(BFILE *)malloc(sizeof(BFILE)))!=NULL) {
		stream->fd=fd;
		stream->mode=md;
		stream->pos=0;
	}
	return stream;
}

ssize_t bread(void *buf, ssize_t size, BFILE *stream)
{
	char *ptr;
	ssize_t more;

	if ((stream==NULL)||(stream->mode!=BMODE_READ)) {
		errno = EBADF;
		return 0;
	}
	more=size;
	ptr=buf;
	while (stream->fill-stream->pos < more) {
		memcpy(ptr,&stream->buf[stream->pos],stream->fill-stream->pos);
		ptr+=stream->fill-stream->pos;
		stream->pos=0;
		if ((stream->fill=read(stream->fd,stream->buf,BBUFSIZ))<=0) {
			stream->eof=(stream->fill==0);
			stream->fill=0;
			return size-more;
		}
	}
	memcpy(ptr,&stream->buf[stream->pos],more);
	stream->pos+=more;
	return size;
}

int beof(BFILE *stream)
{
	return stream->eof;
}

int bclose(BFILE *stream)
{
	int r;
	
	if (stream==NULL) {
		errno = EBADF;
		return BEOF;
	}
	if ((r=close(stream->fd))==0)
		free(stream);
	return r;
}

Voici le fichier bio.h

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
#ifndef _BIO_H_
#define _BIO_H_

#include <sys/types.h>
#include <sys/stat.h>
#include <fcntl.h>

#define BBUFSIZ	1024
#define PERM_FILE (S_IRUSR|S_IWUSR|S_IRGRP|S_IROTH)

#define BEOF (-1)

#define BMODE_READ 'r'
#define BMODE_WRITE 'w'
#define BMODE_RDWR 'b'

typedef struct {
	int fd;
	char mode;
	int eof;
	ssize_t fill;
	ssize_t pos;
	unsigned char buf[BBUFSIZ];
} BFILE;

BFILE *bopen(const char *filename, const char *mode);
ssize_t bread(void *buf, ssize_t size, BFILE *stream);
int beof(BFILE *stream);
int bclose(BFILE *stream);
ssize_t bwrite(void*buf,ssize_t size,BFILE *stream);
int bflush(BFILE*stream);

#endif /* _BIO_H_ */

J'ai surligné en gras les instructions qui je pense pourrait poser problème.Merci de m'aider :

[kamouminator]

je te conceil déja d'initialiser tes pointeurs à null avant toute utiisation:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
BFILE *bopen(const char *filename, const char *mode)
{
	int flags;
	int fd;
	char md;
	BFILE *stream = NULL;
	/* ... */
}

tes "stack smashing" peuvent peut-être être liés à ça...

dans:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
int beof(BFILE *stream)
{
	return stream->eof;
}

tu ne vérifie jamais la validité de stream:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

if(stream == NULL)

[guipe]

Donc dans beof je fais if(stream!= null) blablabla et j'initilaise stream à null dans ma structure.Je vais essayer.