[Resolu] [C Linux] poser un verrou sur un fichier

Recherche :

Mot : Pseudo : Filtrer
Bas de page
Auteur	Sujet : [Resolu] [C Linux] poser un verrou sur un fichier

escaflownn

Bonjour les cracks.

J'ai crée un programme pour me familiariser avec les verrous.
Mon mode opératoire est le suivant, je crée et verrouille un fichier "socket.tmp" dans le repertoire /tmp, puis je mets le programme en pause (getchar).
J'ouvre ce fichier avec l'éditeur vim et je tente d'écrire dessus pour vérifier que le verrou soit bien installé. Or je peux écrire sans problème sur le fichier alors qu'il devrait être vérrouillé.

Si j'ouvre le fichier en lecture seulement, le fcntl() ne veut pas le verrouiller en écriture, ce qui me laisse supposer que fcntl() tente bien la pose de verrou.
La fonction is_locked_socket() censée vérifier que le verrou est présent rapporte dans tous les cas que le fichier n'est pas verrouillé !

=> J'en conclu que fcntl() démarre la pose du verrou mais cette dernière n'aboutie pas. fcntl() ne renvoit pourtant aucun message d'erreur.

main.c

Code :

#include <stdio.h>
#include <stdlib.h>
#include <socket.h>
#include <unistd.h>
const char test[3]="ab";
int main ( int argc , char** argv )
{
int fd;
chdir ( "/tmp" );
if ( ( fd = open_socket ( "socket.tmp" ) ) < 0 )
{
printf ( "Cannot create socket file\n" );
exit ( -1 );
}
printf ( "Press <RETURN> to set lock\n" );
getchar ();
if ( lock_socket ( fd ) < 0 )
{
printf ( "Cannot lock socket file\n" );
exit ( -1 );
}
lseek ( fd , 0 , SEEK_END );
write ( fd , &test[0] , 2 );
printf ( "Press <RETURN> to remove lock\n" );
getchar ();
if ( unlock_socket ( fd ) < 0 )
exit ( -1 );
if ( close_socket ( fd ) < 0 )
exit ( -1 );
exit ( EXIT_SUCCESS );

socket.c

Code :

#include <unistd.h>
#include <sys/types.h>
#include <sys/stat.h>
#include <fcntl.h>
#include "socket.h"
#include <stdio.h>
//--------------------------------------------------------------------
/* Function Name : open_socket
* Object : Opens an existing socket or creates it !
* Input Parameters : [char*] path = Name of the socket file.
* Output Parameters : [int] = File descriptor of the socket file.
* '-1' if error.
* Functions called : open(),lseek().
*/
//--------------------------------------------------------------------
int open_socket ( char* path )
{
int err, fd;
/* OPEN the file */
fd = open ( path , O_RDWR | O_CREAT,
S_IRUSR | S_IWUSR );
if ( fd < 0 )
return SOCKET_ERROR;
/* JUMP at the beginning of the file */
err = lseek ( fd , 0 , SEEK_SET );
if ( err < 0 )
return SOCKET_ERROR;
return fd;
}
//--------------------------------------------------------------------
/* Function Name : close_socket
* Object : Close an existing socket.
* Input Parameters : [int] fd = File descriptor of the socket file.
* Output Parameters : [int] = exit status.
* '-1' if error.
* '0' if no error.
* Functions called : close().
*/
int close_socket ( int fd )
{
return close ( fd );
}
//--------------------------------------------------------------------
/* Function Name : unlock_socket
* Object : Unlock an existing socket.
* Input Parameters : [int] fd = File descriptor of the socket file.
* Output Parameters : [int] = exit status.
* '-1' if error.
* '0' if no error.
* Functions called : fcntl().
*/
//-------------------------------------------------------------------
int unlock_socket ( int fd )
{
static struct flock lock;
lock.l_type = F_UNLCK;
lock.l_whence = SEEK_SET;
lock.l_start = 0;
lock.l_len = 0;
return fcntl ( fd , F_SETLKW , &lock );
}
//--------------------------------------------------------------------
/* Function Name : lock_socket
* Object : Lock an existing socket.
* Input Parameters : [int] fd = File descriptor of the socket file.
* Output Parameters : [int] = exit status.
* '-1' if error.
* '0' if no error.
* Functions called : fcntl().
*/
//-------------------------------------------------------------------
int lock_socket ( int fd )
{
static struct flock lock;
lock.l_type = F_WRLCK;
lock.l_whence = SEEK_SET;
lock.l_start = 0;
lock.l_len = 0;
return fcntl ( fd , F_SETLKW , &lock );
}
//--------------------------------------------------------------------
/* Function Name : is_locked_socket
* Object : Check if an existing socket is locked.
* Input Parameters : [int] fd = File descriptor of the socket file.
* Output Parameters : [int] = exit status.
* the pid of the process if locked.
* '0' if unlocked.
* '-1' if error;
* Functions called : fcntl().
*/
//-------------------------------------------------------------------
int is_locked_socket ( int fd )
{
static struct flock lock;
lock.l_type = F_WRLCK;
lock.l_whence = SEEK_SET;
lock.l_start = 0;
lock.l_len = 0;
if ( fcntl ( fd , F_GETLK , &lock ) == -1 )
return -1;
if ( lock.l_type == F_UNLCK )
return 0;
else
return lock.l_pid;
}

Message cité 1 fois
Message édité par escaflownn le 25-10-2006 à 15:46:37

Publicité

escaflownn

J'ai du mal à croire que personne n'aie jamais posé de verrou Posix ici.

Sve@r

escaflownn a écrit :

Code :

#include <stdio.h>
#include <stdlib.h>
#include <socket.h>
#include <unistd.h>
const char test[3]="ab";
int main ( int argc , char** argv )
{
int fd;
chdir ( "/tmp" );
if ( ( fd = open_socket ( "socket.tmp" ) ) < 0 )
{
printf ( "Cannot create socket file\n" );
exit ( -1 );
}
printf ( "Press <RETURN> to set lock\n" );
getchar ();
if ( lock_socket ( fd ) < 0 )
{
printf ( "Cannot lock socket file\n" );
exit ( -1 );
}
lseek ( fd , 0 , SEEK_END );
write ( fd , &test[0] , 2 );
printf ( "Press <RETURN> to remove lock\n" );
getchar ();
if ( unlock_socket ( fd ) < 0 )
exit ( -1 );
if ( close_socket ( fd ) < 0 )
exit ( -1 );
exit ( EXIT_SUCCESS );

socket.c

Code :

#include <unistd.h>
#include <sys/types.h>
#include <sys/stat.h>
#include <fcntl.h>
#include "socket.h"
#include <stdio.h>
//--------------------------------------------------------------------
/* Function Name : open_socket
* Object : Opens an existing socket or creates it !
* Input Parameters : [char*] path = Name of the socket file.
* Output Parameters : [int] = File descriptor of the socket file.
* '-1' if error.
* Functions called : open(),lseek().
*/
//--------------------------------------------------------------------
int open_socket ( char* path )
{
int err, fd;
/* OPEN the file */
fd = open ( path , O_RDWR | O_CREAT,
S_IRUSR | S_IWUSR );
if ( fd < 0 )
return SOCKET_ERROR;
/* JUMP at the beginning of the file */
err = lseek ( fd , 0 , SEEK_SET );
if ( err < 0 )
return SOCKET_ERROR;
return fd;
}
//--------------------------------------------------------------------
/* Function Name : close_socket
* Object : Close an existing socket.
* Input Parameters : [int] fd = File descriptor of the socket file.
* Output Parameters : [int] = exit status.
* '-1' if error.
* '0' if no error.
* Functions called : close().
*/
int close_socket ( int fd )
{
return close ( fd );
}
//--------------------------------------------------------------------
/* Function Name : unlock_socket
* Object : Unlock an existing socket.
* Input Parameters : [int] fd = File descriptor of the socket file.
* Output Parameters : [int] = exit status.
* '-1' if error.
* '0' if no error.
* Functions called : fcntl().
*/
//-------------------------------------------------------------------
int unlock_socket ( int fd )
{
static struct flock lock;
lock.l_type = F_UNLCK;
lock.l_whence = SEEK_SET;
lock.l_start = 0;
lock.l_len = 0;
return fcntl ( fd , F_SETLKW , &lock );
}
//--------------------------------------------------------------------
/* Function Name : lock_socket
* Object : Lock an existing socket.
* Input Parameters : [int] fd = File descriptor of the socket file.
* Output Parameters : [int] = exit status.
* '-1' if error.
* '0' if no error.
* Functions called : fcntl().
*/
//-------------------------------------------------------------------
int lock_socket ( int fd )
{
static struct flock lock;
lock.l_type = F_WRLCK;
lock.l_whence = SEEK_SET;
lock.l_start = 0;
lock.l_len = 0;
return fcntl ( fd , F_SETLKW , &lock );
}
//--------------------------------------------------------------------
/* Function Name : is_locked_socket
* Object : Check if an existing socket is locked.
* Input Parameters : [int] fd = File descriptor of the socket file.
* Output Parameters : [int] = exit status.
* the pid of the process if locked.
* '0' if unlocked.
* '-1' if error;
* Functions called : fcntl().
*/
//-------------------------------------------------------------------
int is_locked_socket ( int fd )
{
static struct flock lock;
lock.l_type = F_WRLCK;
lock.l_whence = SEEK_SET;
lock.l_start = 0;
lock.l_len = 0;
if ( fcntl ( fd , F_GETLK , &lock ) == -1 )
return -1;
if ( lock.l_type == F_UNLCK )
return 0;
else
return lock.l_pid;
}

Désolé, j'ai pas le temps d'examiner ton code en détail. Je remarque quand-même que les fonctions "lock_socket" et "unlock_socket" sont quasiment les mêmes et pourraient être regroupées en une seule qui recevrait un paramètre supplémentaire "lock/unlock". Mais le pb ne vient pas de là.
Télécharge mon cours sur les processus http://fr.lang.free.fr/cours/Processus_Csyst_v1.0.pdf. Il y a un chapitre qui parle de la pose des verrous. Si j'ai un peu de temps, je reviendrai plus tard voir ce qui peut clocher...

Petit conseil: tu devrais utiliser de temps en temps "errno" ainsi que "perror()" histoire d'afficher ce que te dit le système quand ça cloche.
Autre remarque: Tes fonctions "lock/unlock" utilisent une structure statique ce qui signifie que la variable "lock" garde ses valeurs entre chaque appel. Si jamais un des membres de "lock" n'est pas réinitialisé alors qu'il le devrait, il gardera la valeur précédente et si celle-ci n'est pas bonne...
En général, une variable locale ne se met "static" que si on a un besoin impératif de garder sa valeur entre chaque appel (cas d'un compteur d'appels par exemple)...

Message édité par Sve@r le 25-10-2006 à 15:20:50

---------------
Vous ne pouvez pas apporter la prospérité au pauvre en la retirant au riche.

escaflownn

Merci pour tes précieux conseils, j'ai refondu le code en conséquence.

Concernant le problème, l'erreur venait du mode opératoire en deux points:
*tout d'abord vim ne tient pas compte des verrous, pour vérifier que le verrou est posé, il suffit en réalité d'éxécuter deux fois le processus en même temps.
*la fonction is_locked_socket() ne permet pas de vérifier à postériori si le processus a lui même posé un verrou, car un processus peut toujours poser un verrou à la place d'un verrou qu'il a lui même posé antèrieurement. C'est pour cette raison qu'elle renvoyait 0.

main.c

Code :

#include <stdio.h>
#include <stdlib.h>
#include <unistd.h>
#include <errno.h>
#include <sys/types.h>
#include <sys/stat.h>
#include <fcntl.h>
#include "lock.h"
int main ( int argc , char** argv )
{
int fd;
char tab[4] = "ab\n";
if ( chdir ( "/tmp" ) < 0 )
{
perror ( "[main.c] chdir(0)" );
exit ( EXIT_FAILURE );
}
fd = open ( "fic_test.tmp" , O_RDWR | O_CREAT , S_IRWXU );
if ( fd < 0 )
{
perror ( "[main.c] open(0)" );
exit ( EXIT_FAILURE );
}
if ( lock_file ( fd , LOCK_NOWAIT ) < 0 )
{
perror ( "[main.c] lock_file(0)" );
exit ( EXIT_FAILURE );
}
if ( lseek ( fd , 0 , SEEK_END ) < 0 )
{
perror ( "[main.c] lseek(0)" );
exit ( EXIT_FAILURE );
}
printf ( "Press <RETURN> to continue\n" );
getchar ( );
if ( write ( fd , &tab[0] , 3 ) < 0 )
{
perror ( "[main.c] write(0)" );
exit ( EXIT_FAILURE );
}
if ( lock_file ( fd , LOCK_UNLOCK ) < 0 )
{
perror ( "[main.c] lock_file(1)" );
exit ( EXIT_FAILURE );
}
if ( close ( fd ) < 0 )
{
perror ( "[main.c] close(0)" );
exit ( EXIT_FAILURE );
}
exit ( EXIT_SUCCESS );
}

lock.c

Code :

//--------------------------------------------------------------------
//--------------------------------------------------------------------
/* Lib Name : file.c
* Ver : 1.0
* Description : Provide basic functions to
* manipulate files.
*/
//--------------------------------------------------------------------
#include <fcntl.h>
#include <stdio.h>
#include <errno.h>
#include "lock.h"
//--------------------------------------------------------------------
/* Function Name : lock_file
* Object : Lock an opened file.
* Input Parameters : [int] fd = File descriptor of the file.
* [int] cmd = kind of lock to apply.
* Output Parameters : [int] = - '-1' if error.
* - '0' if no error.
* Functions called : fcntl().
*/
//-------------------------------------------------------------------
int lock_file ( int fd , int cmd )
{
struct flock lock;
/* Setup an exclusive (writing) lock */
lock.l_whence = SEEK_SET;
lock.l_start = 0;
lock.l_len = 0;
/* Apply the lock */
if ( cmd == LOCK_NOWAIT )
{
lock.l_type = F_WRLCK;
return fcntl ( fd , F_SETLK , &lock );
}
else if ( cmd == LOCK_WAIT )
{
lock.l_type = F_WRLCK;
return fcntl ( fd , F_SETLKW , &lock );
}
else if ( cmd == LOCK_UNLOCK )
{
lock.l_type = F_UNLCK;
return fcntl ( fd , F_SETLK , &lock );
}
else
{
/* Invalid argument */
errno = EINVAL;
return -1;
}
}
//--------------------------------------------------------------------
/* Function Name : lock_is_set
* Object : Check if an opened file is locked
* by AN OTHER processus.
* Input Parameters : [int] fd = File descriptor of the file.
* Output Parameters : [int] = - the pid of the process which locked it.
* - '0' if unlocked.
* - '-1' if error;
* Functions called : fcntl().
*/
//-------------------------------------------------------------------
int lock_is_set ( int fd )
{
struct flock lock;
/* Test an exclusive (writing) lock */
lock.l_type = F_WRLCK;
lock.l_whence = SEEK_SET;
lock.l_start = 0;
lock.l_len = 0;
/* Apply the lock and check the result */
if ( fcntl ( fd , F_GETLK , &lock ) < 0 )
return -1;
else if ( lock.l_type == F_UNLCK )
return 0;
else
/* lock.l_pid is now the pid of the process
* owner of the lock */
return lock.l_pid;
}

Message cité 1 fois
Message édité par escaflownn le 25-10-2006 à 16:57:14

Sve@r

escaflownn a écrit :

Merci pour tes précieux conseils.

J'ai pas dit grand chose de réellement utile. Mais si ça marche, tant mieux.

Message édité par Sve@r le 25-10-2006 à 21:18:25

---------------
Vous ne pouvez pas apporter la prospérité au pauvre en la retirant au riche.

FORUM HardWare.fr

Programmation

[Resolu] [C Linux] poser un verrou sur un fichier

Sujets relatifs
[Résolu] Problème de clic sur textarea ?	Conception Client/Serveur (résolu)
[Oracle] - Limitation des varchar2 [Resolu]	[Résolu] Récupération automatique de fichier sur Internet
[Résolu]Activer le code html sur un forum ? (et le desactiver ici lol)	[Résolu][XHTML/Javascript/W3C] Problème de XHTML dans un Javascript
C# et application tierce (MS-EXCEL) (résolu)	[Résolu] Changement de couleur au passage de la souris
[Résolu]Problème de caractères depuis migration de siteweb
Plus de sujets relatifs à : [Resolu] [C Linux] poser un verrou sur un fichier

Page générée en 0.099 secondes