"error LNK2005: " à cause d'une fonction

Version imprimable

Hello

Dans un header nommé documented.h j'ai ecris la definition suivante:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
 
VOID NTAPI HalReadWritePCISpace
(
  IN ULONG   BusNumber,
  IN ULONG   SlotNumber,
  IN ULONG   RegisterNumber,
  IN PVOID   Buffer,
  IN ULONG   Length,
  IN BOOLEAN WritePCISpace
)
{
    return;
}

Dès lors des que j'inclue dans un entête documented.h j'ai ce message et je comprend pas:

Code:

1
2
3
 
pci.obj : error LNK2005: "void __stdcall HalReadWritePCISpace(unsigned long,unsigned long,unsigned long,void *,unsigned long,unsigned char)" (?HalReadWritePCISpace@@YGXKKKPAXKE@Z) already defined in main.obj
Debug/test.exe : fatal error LNK1169: one or more multiply defined symbols found

Auriez vous une solution SVP?

Merci

11/01/2020, 14h38
koala01

Salut,

N'aurais tu pas, par le plus grand des hasards, inclus le fichier d'implémentation (*.cpp) qui contient cette fonction dans ton fichier main.cpp :question:

N'as tu pas encore compris qu'il ne fallait JAMAIS inclure un fichier .cpp, qu'il n'y avait que les fichier d'en-tête qui devait être utilisés avec la directive #include :question:
11/01/2020, 15h11
dalfab

Et il ne faut jamais définir de fonction dans un fichier header.
A chaque inclusion du header la fonction sera créée et recréée, ce qui est l'erreur que tu obtiens.

Merci

J'ai inclu aucun .cpp nul part parcontre je n'ai fait dans documented.h qu'une définition. Du coup j'ai essayé de faire une déclaration dans le documented.h et à la compilation aucun souci:

Code:

1
2
3
4
5
6
7
8
9
10
 
VOID NTAPI HalReadWritePCISpace
(
  IN ULONG   BusNumber,
  IN ULONG   SlotNumber,
  IN ULONG   RegisterNumber,
  IN PVOID   Buffer,
  IN ULONG   Length,
  IN BOOLEAN WritePCISpace
);

Par contre dès que je créer le documented.cpp et que j'inclue le documented.h j'ai un paquet d'erreur sans même avoir écrit une ligne:

Code:

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
 
 
Compiling...
undocumented.cpp
c:\...\documented.h(33) : error C2146: syntax error : missing ';' before identifier 'NTSTATUS'
c:\...\documented.h(33) : error C2501: 'NTSTATUS' : missing storage-class or type specifiers
c:\...\documented.h(43) : error C2146: syntax error : missing ';' before identifier 'SHORT'
c:\...\documented.h(43) : error C2146: syntax error : missing ';' before identifier 'CSHORT'
c:\...\documented.h(43) : error C2501: 'SHORT' : missing storage-class or type specifiers
c:\...\documented.h(43) : error C2501: 'CSHORT' : missing storage-class or type specifiers
c:\...\documented.h(47) : error C2143: syntax error : missing ';' before '*'
c:\...\documented.h(47) : error C2501: 'PSZ' : missing storage-class or type specifiers
c:\...\documented.h(48) : error C2143: syntax error : missing ';' before '*'
c:\...\documented.h(48) : error C2501: 'PCSZ' : missing storage-class or type specifiers
c:\...\documented.h(53) : error C2146: syntax error : missing ';' before identifier 'Length'
c:\...\documented.h(53) : error C2501: '_STRING::USHORT' : missing storage-class or type specifiers
c:\...\documented.h(53) : error C2501: '_STRING::Length' : missing storage-class or type specifiers
c:\...\documented.h(54) : error C2146: syntax error : missing ';' before identifier 'MaximumLength'
c:\...\documented.h(54) : error C2501: '_STRING::USHORT' : missing storage-class or type specifiers
c:\...\documented.h(54) : error C2501: '_STRING::MaximumLength' : missing storage-class or type specifiers
c:\...\documented.h(55) : error C2146: syntax error : missing ';' before identifier 'Buffer'
c:\...\documented.h(55) : error C2501: '_STRING::PCHAR' : missing storage-class or type specifiers
c:\...\documented.h(55) : error C2501: '_STRING::Buffer' : missing storage-class or type specifiers
c:\...\documented.h(67) : error C2146: syntax error : missing ';' before identifier 'Status'
c:\...\documented.h(67) : error C2501: '_IO_STATUS_BLOCK::__unnamed::NTSTATUS' : missing storage-class or type specifiers
c:\...\documented.h(67) : error C2501: '_IO_STATUS_BLOCK::__unnamed::Status' : missing storage-class or type specifiers
c:\...\documented.h(68) : error C2146: syntax error : missing ';' before identifier 'Pointer'
c:\...\documented.h(68) : error C2501: '_IO_STATUS_BLOCK::__unnamed::PVOID' : missing storage-class or type specifiers
c:\...\documented.h(68) : error C2501: '_IO_STATUS_BLOCK::__unnamed::Pointer' : missing storage-class or type specifiers
c:\...\documented.h(71) : error C2146: syntax error : missing ';' before identifier 'Information'
c:\...\documented.h(71) : error C2501: '_IO_STATUS_BLOCK::ULONG_PTR' : missing storage-class or type specifiers
c:\...\documented.h(71) : error C2501: '_IO_STATUS_BLOCK::Information' : missing storage-class or type specifiers
c:\...\documented.h(77) : error C2065: 'NTAPI' : undeclared identifier
c:\...\documented.h(77) : error C2065: 'PIO_APC_ROUTINE' : undeclared identifier
c:\...\documented.h(77) : error C2501: 'VOID' : missing storage-class or type specifiers
c:\...\documented.h(77) : fatal error C1903: unable to recover from previous error(s); stopping compilation

J'ai ces erreurs que dans ce cas là.

11/01/2020, 15h39
dalfab

Tu nous indiques un fichier de 10 lignes qui a des erreurs qui apparaissent de la ligne 33 à la ligne 77, ça ne pas facile du tout pour pouvoir t'aider!

Au feeling, je dirais qu'il manque des #include dans le fichier documented.h.
11/01/2020, 15h45
Nono Sto

En fait pour l'instant tant que je definis dans documented.h toute les fonctions pas de probleme mais si je fais sa dans un .cpp j'ai des erreur.
11/01/2020, 18h34
koala01

Citation:

Envoyé par dalfab

Tu nous indiques un fichier de 10 lignes qui a des erreurs qui apparaissent de la ligne 33 à la ligne 77, ça ne pas facile du tout pour pouvoir t'aider!

Au feeling, je dirais qu'il manque des #include dans le fichier documented.h.

En fait, il y a de très fortes chances pour que le compilateur ait en réalité "perdu les pédales" à la ligne 33, lorsqu'il n'a pas compris ce que représentait NTSTATUS (qui n'apparait d'ailleurs pas dans le code montré :P), et que les autres erreurs ne soient que des "faux positifs" induits par cette "perte des pédales".

Mais, de fait, je dirais sensiblement la même chose au feeling :D
11/01/2020, 18h49
koala01
Citation:

Envoyé par Nono Sto

En fait pour l'instant tant que je definis dans documented.h toute les fonctions pas de probleme mais si je fais sa dans un .cpp j'ai des erreur.

Ben oui, mais il y a des règles pour définir les fonctions dans un fichier d'en-tête...

La principale d'entre-elles étant que tu dois t'assurer que l'éditeur de liens considérera qu'il est "normal" de se retrouver avec plusieurs implémentations (au niveau du code binaire généré) de la même fonction; par exemple, en la déclarant inline.

Et, de toutes manières, le fait de déporter la définition d'une fonction dans un fichier d'implémentation ne devrait pas avoir de répercussion sur l'analyse du fichier d'en-tête dans lequel elle se trouvait à l'origine.

"Dans le pire des cas", l'implémentation déportée sera dans un fichier qui ne sera pas compilé à cause d'un problème de définition du projet, ce qui se traduira par une erreur lors de l'édition de liens.

Si bien que, au vu des deux situations dont nous avons connaissance, à savoir:
1. l'implémentation d'une fonction dans le fichier d'en-tête qui compile mais provoque une erreur à l'édition de liens et
2. l'implémentation d'une fonction dans un fichier d'implémentation qui provoque une erreur de compilation lors de l'analyse du fichier d'en-tête
je suis désolé d'avoir à te le dire, mais il n'y a qu'une seule conclusion à tirer: tu as de toutes évidence fait "une connerie" en déportant la fonction qui posait problème, sans doute en supprimant l'inclusion d'un fichier d'en-tête que tu auras peut-être jugé (à tors) comme inutile ;)
11/01/2020, 20h01
Nono Sto

Le problème est que j'ai du récupérer ce fameux documented.h et il est délicat à modifier. Du coup la meilleur solution est d'ajouter les définitions des quelques fonctions que je dois ajouter dans le documented.h. Alors quels conseil me donneriez vous SVP pour essayé de faire sa propre?
12/01/2020, 02h34
Bousk

Tu utilises quel compilateur ?
Est-il trop stupide pour redéfinir la fonction à chaque include plutôt que de l'inliner ?
Puisque tu implémentes la fonction dans un fichier header, pourquoi n'est-elle pas inline ?
14/01/2020, 10h56
Médinoc

Je sais qu'on dit parfois que lorsqu'on écrit une implémentation, il est de bon ton de commencer par le #include spécifique, mais dans le cas d'un dev Windows, je pense que commencer par <windows.h> (ou si l'on utilise des en-têtes précompilés, "stdafx.h") prend le pas sur cette règle de style.