Re: [Spip-zone-commit] r29591 - /_plugins_/clevermail/2_0/action/clevermail_list_subscriber_toggle.php
cedric.morin <at> yterium.com <cedric.morin <at> yterium.com>
2009-07-02 16:13:05 GMT
Le 2 juil. 09 à 17:59, Nicolas Hoizey a écrit :
Le 2 juil. 09 à 15:39, cedric.morin <at> yterium.com a écrit :
sql_quote alors car là ce n'est plus protégé !
Mais c'est pas le but du "q" à la fin de "sql_updateq" justement ???
oui mais y a pas de q dans
sql_getfetsel("lsr_mode", "spip_cm_lists_subscribers", "lsr_id=".$lsr_id);
sql_countsel("spip_cm_lists_subscribers", "lsr_id=".$lsr_id)
...
Une bonne règle que j'applique systématiquement est de *toujours* protéger chaque variable PHP qui apparait dans un sql_xx, et ce, même si la variable a subi un intval 2 lignes plus tôt.
Cela permet de voir immédiatement qu'une requête est sûre, sans avoir à lire le code en amont.
Et ça ouvre la porte à des outils de verif automatisés.
Cédric
<div>
<br><div>
<div>Le 2 juil. 09 à 17:59, Nicolas Hoizey a écrit :</div>
<br class="Apple-interchange-newline"><blockquote type="cite"><span class="Apple-style-span">Le 2 juil. 09 à 15:39,<span class="Apple-converted-space"> </span><a href="mailto:cedric.morin <at> yterium.com">cedric.morin <at> yterium.com</a><span class="Apple-converted-space"> </span>a écrit :<br><blockquote type="cite">sql_quote alors car là ce n'est plus protégé !<br>
</blockquote>
<br>Mais c'est pas le but du "q" à la fin de "sql_updateq" justement ???</span></blockquote>
</div>
<br><div>oui mais y a pas de q dans</div>
<div>
<blockquote type="cite"><blockquote type="cite">sql_getfetsel("lsr_mode", "spip_cm_lists_subscribers", "lsr_id=".$lsr_id);</blockquote></blockquote>
<blockquote type="cite"><blockquote type="cite">sql_countsel("spip_cm_lists_subscribers", "lsr_id=".$lsr_id)</blockquote></blockquote>...</div>
<div>Une bonne règle que j'applique systématiquement est de *toujours* protéger chaque variable PHP qui apparait dans un sql_xx, et ce, même si la variable a subi un intval 2 lignes plus tôt.</div>
<div>Cela permet de voir immédiatement qu'une requête est sûre, sans avoir à lire le code en amont.</div>
<div>Et ça ouvre la porte à des outils de verif automatisés.</div>
<div><br></div>
<div>Cédric</div>
</div>