différence entre strmov et strcpy ?

Recherche :

Mot : Pseudo : Filtrer
Bas de page
Auteur	Sujet : différence entre strmov et strcpy ?

joce

Architecte / Développeur principal
"BugHunter"

apparemment un strmov fait ca :

Code :

char *strmov(register char *dst, register const char *src)
{
while ((*dst++ = *src++)) ;
return dst-1;
}

quelle différence avec strcpy ? (hormis que strmov fait pointer vers le \0 final)

Message édité par joce le 29-03-2005 à 02:45:42

Publicité

el muchacho

Comfortably Numb

"La fonction strmov() utilisée dans cet exemple est inclue dans la bibliothèque mysqlclient et fonctionne comme strcpy() mais retourne un pointeur sur le nul de fin du premier paramètre. Valeur de retour La longueur de la valeur passée dans to , n'incluant pas la caractère nul de fin de chaîne."
http://www.nexen.net/docs/mysql/an [...] string.php

Message édité par el muchacho le 29-03-2005 à 02:57:30

joce

Architecte / Développeur principal
"BugHunter"

mais est-ce que src est "déplacé" dans dst en pratique :??:

et qu'est ce qui se passe si src est effacé après coup ? (a priori rien ?)

Je demande ca aussi par rapport à la version assembler qui semble faire un movb pour chaque caractère

Code :

00045 char *strmov(dst, src)
00046 char *dst, *src;
00047 {
00048 asm(" movl 4(a7),a1 " );
00049 asm(" movl 8(a7),a0 " );
00050 asm(".L4: movb (a0)+,(a1)+ " );
00051 asm(" jne .L4 " );
00052 asm(" movl a1,d0 " );
00053 asm(" subql #1,d0 " );
00054 }
00055

Message édité par joce le 29-03-2005 à 02:57:01

el muchacho

Comfortably Numb

J'ai édité mon précédent post.

joce

Architecte / Développeur principal
"BugHunter"

c'est justement dans le cadre de mysql que je m'y interesse

Code :

void close_temporary_tables(THD *thd)
{
TABLE *table,*next;
char *query, *end;
uint query_buf_size;
bool found_user_tables = 0;
if (!thd->temporary_tables)
return;
LINT_INIT(end);
query_buf_size= 50; // Enough for DROP ... TABLE IF EXISTS
for (table=thd->temporary_tables ; table ; table=table->next)
/*
We are going to add 4 ` around the db/table names, so 1 does not look
enough; indeed it is enough, because table->key_length is greater (by 8,
because of server_id and thread_id) than db||table.
*/
query_buf_size+= table->key_length+1;
if ((query = alloc_root(thd->mem_root, query_buf_size)))
// Better add "if exists", in case a RESET MASTER has been done
end=strmov(query, "DROP /*!40005 TEMPORARY */ TABLE IF EXISTS " );
for (table=thd->temporary_tables ; table ; table=next)
{
if (query) // we might be out of memory, but this is not fatal
{
// skip temporary tables not created directly by the user
if (table->real_name[0] != '#')
found_user_tables = 1;
/*
Here we assume table_cache_key always starts
with \0 terminated db name
*/
end = strxmov(end,"`",table->table_cache_key,"`.`",
table->real_name,"`,", NullS);
}
next=table->next;
close_temporary(table);
}
if (query && found_user_tables && mysql_bin_log.is_open())
{
/* The -1 is to remove last ',' */
thd->clear_error();
Query_log_event qinfo(thd, query, (ulong)(end-query)-1, 0, FALSE);
/*
Imagine the thread had created a temp table, then was doing a SELECT, and
the SELECT was killed. Then it's not clever to mark the statement above as
"killed", because it's not really a statement updating data, and there
are 99.99% chances it will succeed on slave.
If a real update (one updating a persistent table) was killed on the
master, then this real update will be logged with error_code=killed,
rightfully causing the slave to stop.
*/
qinfo.error_code= 0;
mysql_bin_log.write(&qinfo);
}
thd->temporary_tables=0;
}

apparement cette fonction crash dans certains cas, quand un thread se termine et qu'on doit fermer les temp table rattachée à la connexion (et donc marquer DROP /*!40005 TEMPORARY */ TABLE IF EXISTS ... dans le binlog).

Me demande si c'est un problème d'estimation de la taille de query_buf_size ou un autre problème

Ce qui est bizarre c'est que si dans le thread y a des users temp table et des temp table "internes", il va quand même écrire dans le binlog le DROP TABLE sur la table interne :heink:
C'est louche

Message édité par joce le 29-03-2005 à 03:10:53

el muchacho

Comfortably Numb

Je doute que ça plante dans le strmov vu que 50 char son alloués au minimum. Par contre je verrais bien un plantage dans le strxmov qui suit.
Le query_buf_size+= table->key_length+1;
alors que strxmov (google) concatène plus de choses, à savoir :

# end = strxmov(end,"`",table->table_cache_key,"`.`",
# table->real_name,"`,", NullS);

soit 3 caractères, le cache_key et le real_name.
Donc si cette chaine fait moins de 50+table->key_length+1, ça passe, sinon ça casse.

Le code correct serait plutôt :
query_buf_size+= table->key_length + table->real_name_length + 6;

avec une longueur initiale de 22 octets (pour le strmov).
Autre chose, le commentaire "// we might be out of memory, but this is not fatal", semblant signifier que alloc_root ne retourne pas forcément la mémoire demandée, auquel cas le strxmov derrière peut se ramasser.
Si c'est le cas, pour moi, ça ferait 2 bugs potentiels à vérifier.
Enfin, en question subsidiaire, vérifier le comportement de strxmov en cas d'overlapping entre src et dst.

Message édité par el muchacho le 01-04-2005 à 09:12:46

Sve@r

joce a écrit :

apparemment un strmov fait ca :

Code :

char *strmov(register char *dst, register const char *src)
{
while ((*dst++ = *src++)) ;
return dst-1;
}

quelle différence avec strcpy ? (hormis que strmov fait pointer vers le \0 final)

Déjà ce n'est pas tout à fait correct. Si la fonction est vraiment écrite comme ça alors elle va foirer si on l'appelle de cette façon

strmov(ptr + 1, ptr)

Elle recopiera le premier octet de "src" à l'infini.

Pour qu'elle fonctionne correctement, elle doit d'abord regarder les positions de "src" et "dst". Si "dst" est placé après "src", elle doit alors balayer "src" à reculon pour que ça marche à 100%. Si "dst" est placé avant "src", alors elle doit balayer "src" normallement.

Message édité par Sve@r le 29-03-2005 à 10:52:53

Taz

bisounours-codeur

restrict powa

matafan

Notez que d'apres la description que vous en donnez, strmov fait la meme chose que stpcpy, qui est disponible sur de nombreux systeme (mais ne fais partie d'aucun standard).

joce

Architecte / Développeur principal
"BugHunter"

el muchacho a écrit :

T'as pas du faire attention au commentaire :

# /*
# We are going to add 4 ` around the db/table names, so 1 does not look
# enough; indeed it is enough, because table->key_length is greater (by 8,
# because of server_id and thread_id) than db||table.
# */

FORUM HardWare.fr

Programmation

différence entre strmov et strcpy ?

Sujets relatifs
Différence entre class et id ?	différence fondamentale entre le modèle objet et le modèle relationnel
&# x92; et &# x92; ::: oui je sais il n'y a aucune difference.	Différence d'affichage entre IE et Firefox avec du XHTML strict
strcpy?? c flou.... besoin d'aide.. merci	Différence de comportement sur 2 environnements
Différence entre les compilateurs?	difference de rendu de pages web
Différence entre SHOW COLUMN et mysql_fetch_field	difference php, php3, php4 ? [NewBie]
Plus de sujets relatifs à : différence entre strmov et strcpy ?

Page générée en 0.066 secondes