La fonction callback dans c++

**bboy0303** · 23/05/2009, 12h38

J' ai une classe DialogImpl:

Code :

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

1
2
3
4
5
6
7
8
9
class DialogImpl
{
public:
    void my_callback(u_char *useless, const struct pcap_pkthdr* hdr, const u_char* packet);
public slots:
    void sniffer();
 
    // ............ les autres méthode
}

Ma méthode sniffer() a cette fonction:

Code :

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

1
2
3
4
5
DialogImpl::sniffer() {
    //............
    pcap_loop(descr,-1,my_callback,NULL);
    //................
}

Si je fais comme dessus, il va créer une erreur argument of type ‘void (DialogImpl:: )(u_char*, const pcap_pkthdr*, const u_char*)’ does not match ‘void ( * )(u_char*, const pcap_pkthdr*, const u_char*)’

Je sais qu'il faut mettre static à la fonction my_callback. Pourtant, je ne peux pas utiliser les fonctions de GUI dans my_callback. Est-ce qu' il y a une autre résolution?

Merci par avance pour votre aide!

**jfouche** · 23/05/2009, 16h45

Salut

Code :

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

pcap_loop(descr,-1,my_callback,NULL);

Je suppose que le dernier parametre (NULL) est le parametre de la fonction callback (c'est souvent le cas). Tu peux donc créer une fonction static :

Code :

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

1
2
3
4
5
6
7
8
9
 
static
void MaBelleCallback(u_char *data, const struct pcap_pkthdr* hdr, const u_char* packet)
{
    DialogImpl* pDlg = static_cast<data*>(data);
    assert(pDlg);
    pDlg->my_callback(hdr, packet);
 
}

Il faut donc modifier ton code en conséquence :

Code :

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

1
2
3
4
5
DialogImpl::sniffer() {
    ...
    pcap_loop(descr,-1,my_callback,this);
    ...
}

et

Code :

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

1
2
3
4
5
6
class DialogImpl
{
public:
    void my_callback(const struct pcap_pkthdr* hdr, const u_char* packet);
    ...
}

En espérant que ça répond à ta question.
--
Jérémie

**bboy0303** · 23/05/2009, 19h30

Merci bien, j'ai réussi à compiler le programme mais il y a une telle erreur dans la fonction pcap_loop que mon programme ne peut pas éxécuter normalement, je vais rechercher encore!

**jfouche** · 23/05/2009, 20h36

Envoyé par jfouche

Il faut donc modifier ton code en conséquence :

Code :

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

1
2
3
4
5
DialogImpl::sniffer() {
    ...
    pcap_loop(descr,-1,my_callback,this);
    ...
}

Je me suis trompé, c'est :

Code :

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

1
2
3
4
5
DialogImpl::sniffer() {
    ...
    pcap_loop(descr,-1,MaBelleCallback,this);
    ...
}

--
Jérémie

**bboy0303** · 24/05/2009, 09h53

Merci, je sais!
Il faut corriger aussi pour faire une forçage de type:

Code :

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

	pcap_loop(descr,-1,MaBelleCallback,(u_char*)this);

Mais ton code a fait quelques erreurs en compilant:

Code :

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

1
2
3
4
5
6
static void MaBelleCallback(u_char *data, const struct pcap_pkthdr* hdr, const u_char* packet)
{
    DialogImpl* pDlg = static_cast<data*>(data);
    assert(pDlg);
    pDlg->my_callback(hdr, packet);
}

src/dialogimpl.h:29: erreur: expected type-specifier before ‘data’
src/dialogimpl.h:29: erreur: expected `>' before ‘data’
src/dialogimpl.h:29: erreur: expected `(' before ‘data’
src/dialogimpl.h:29: erreur: expected primary-expression before ‘>’ token
src/dialogimpl.h:29: erreur: expected `)' before ‘;’ token

J' ai fais comme-ci, il peut être compiler mais le programme ne marche pas encore normalement:

Code :

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

1
2
3
4
5
static void MaBelleCallback(u_char *data, const struct pcap_pkthdr* hdr, const u_char* packet)
{
    DialogImpl* pDlg = (DialogImpl*)data;
    pDlg->my_callback(hdr, packet);
}

Tu connais une résolution?

**jfouche** · 24/05/2009, 11h03

Utilises ton debuger pour voir ce qui passe mal.
Passes tu bien dans la MaBelleCallback ?
--
Jérémi

**bboy0303** · 24/05/2009, 11h27

Selon moi, il ne passe pas MaBellCallback.
Quand je n'utilise pas la classe, ca marche bien. Mais avec la classe, ca se passe mal. Mon code avec la classe:
dialogimpl.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
#ifndef DIALOGIMPL_H
#define DIALOGIMPL_H
//
#include <pcap.h>
#include <stdio.h>
#include <stdlib.h>
#include <string>
#include <errno.h>
#include <sys/socket.h>
#include <netinet/in.h>
#include <arpa/inet.h>
#include <netinet/if_ether.h>
#include <netinet/ip.h>
#include <netinet/tcp.h>
#include <netinet/udp.h>
#include <iostream>
#include <fstream>
using namespace std;
 
class DialogImpl
{
public:
	static void my_callback(u_char *useless, const struct pcap_pkthdr* hdr, const u_char* packet) {
		DialogImpl *This = (DialogImpl*)useless;
		This->got_packet(hdr, packet);
	}
 
	void got_packet(const struct pcap_pkthdr* hdr, const u_char* packet);
 
	void sniffer();
};
#endif

dialogimpl.cpp:

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
#include "dialogimpl.h"
 
void DialogImpl::got_packet(const struct pcap_pkthdr* hdr, const u_char* packet) {
	cout << "Hello";
}
 
 
void DialogImpl::sniffer()
{
	char *dev;
	char errbuf[PCAP_ERRBUF_SIZE];
	pcap_t *descr;
 
	cout << "Start\n";
 
	dev = pcap_lookupdev(errbuf);
	if(dev == NULL) { 
		printf("%s\n",errbuf); 
		exit(1); 
	}
 
	descr = pcap_open_live(dev,BUFSIZ,0,-1,errbuf);
	if(descr == NULL) { 
		printf("pcap_open_live(): %s\n",errbuf); 
		exit(1); 
	}
 
	pcap_loop(descr,-1,my_callback,(u_char*)this);
	pcap_close(descr);
}

main.cpp:

Code :

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

1
2
3
4
5
6
7
8
#include "dialogimpl.h"
 
int main()
{
	DialogImpl win;
	win.sniffer();
	return 0;
}

Pourrais-tu m'aider?

**bboy0303** · 24/05/2009, 12h39

Si je n'utilise pas la classe, ca marche bien:
foo.cpp:

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
#include <pcap.h>
#include <stdio.h>
#include <stdlib.h>
#include <string>
#include <errno.h>
#include <sys/socket.h>
#include <netinet/in.h>
#include <arpa/inet.h>
#include <netinet/if_ether.h>
#include <netinet/ip.h>
#include <netinet/tcp.h>
#include <netinet/udp.h>
#include <iostream>
#include <fstream>
using namespace std;
 
void my_callback(u_char *useless,const struct pcap_pkthdr* hdr,const u_char* packet)
{	
	cout << "Hello\n";
}
 
int main()
{
	char *dev;
	char errbuf[PCAP_ERRBUF_SIZE];
	pcap_t *descr;
 
	cout << "Start\n";
 
	dev = pcap_lookupdev(errbuf);
	if(dev == NULL) { 
		printf("%s\n",errbuf); 
		exit(1); 
	}
 
	descr = pcap_open_live(dev,BUFSIZ,0,-1,errbuf);
	if(descr == NULL) { 
		printf("pcap_open_live(): %s\n",errbuf); 
		exit(1); 
	}
 
	pcap_loop(descr,-1,my_callback,NULL);
	pcap_close(descr);
   	return 0;
}

Compiler et éxécuter:
g++ foo.cpp -lpcap
sudo ./a.out

**jfouche** · 24/05/2009, 13h13

Je n'aurai pas mis my_callback en membre static de DialogImpl, mais plutôt une fonction static toute seule dans son coin.

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
#ifndef DIALOGIMPL_H
#define DIALOGIMPL_H
...
// As tu besoin d'inclure tous ces header dans ton .h ?
// ne défini pas de using namespace dans un .h, qui sais si l'utilisateur
// de ta classe n'en veux pas ?
// using namespace std; 
 
class DialogImpl
{
public:
	void got_packet(const struct pcap_pkthdr* hdr, const u_char* packet);
 
	void sniffer();
};
#endif

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
#include "dialogimpl.h"
 
// Ici tu peux, sans gener personne
using namespace std; 
 
static void my_callback(u_char *data, const struct pcap_pkthdr* hdr, const u_char* packet)
{
	cout << "my_callback(...)" << endl;
	DialogImpl *This = (DialogImpl*)data;
	This->got_packet(hdr, packet);
}
 
void DialogImpl::got_packet(const struct pcap_pkthdr* hdr, const u_char* packet)
{
	cout << "Hello";
}
 
void DialogImpl::sniffer()
{
	char *dev;
	char errbuf[PCAP_ERRBUF_SIZE];
	pcap_t *descr;
 
	cout << "Start" << endl;
 
	dev = pcap_lookupdev(errbuf);
	if(dev == NULL) {
		cerr << errbuf << endl; 
		exit(1); 
	}
 
	descr = pcap_open_live(dev,BUFSIZ,0,-1,errbuf);
	if(descr == NULL) { 
		cerr << "pcap_open_live(): " << errbuf << std::endl; 
		exit(1); 
	}
 
	pcap_loop(descr,-1,my_callback,(u_char*)this);
	pcap_close(descr);
}