[php] Verwijder probleem

[ultimatia]

Hallo mensen,

Ik zit met het volgende probleem;
In een soort van admin schermpje kan ik cd`s o.a wijzigen en verwijderen.
Nu krijg ik bij verwijderen wel een waarschuwingsmelding als ik zeker weet als ik het wil verwijderen, maar waneer ik hier daadwerkelijk voor kies wordt er niets verwijderd.

De twee betreffende php bestanden zijn de volgende;
Admin_CD

PHP:

<?php 
    
    include ('php_header.php');
//  include ('beveiliging.php');
    include ('html_header.php');

?>  

    <script language="Javascript">
    function submit_cd(CD_ID, type) {
       
      document.forms.cd.CD_ID.value = CD_ID;
      if(type == 'verwijderen') {
        document.forms.cd.action = 'cd_verwijderen.php';
        document.forms.cd.submit();
      }
      if(type == 'wijzigen') {
        document.forms.cd.action = 'cd_wijzigen.php';
        document.forms.cd.submit();
      }
      
      
      return true;
    }
    </script>
    
    <div id="content">
    
    <p> <a href="cd_toevoegen.php">klik hier om een cd toe te voegen.</a><br />
    
    
    <form method="post" action="cd.php" name="cd">
    <input type="hidden" name="CD_ID"/>
    <input type="hidden" name="type"/>
    </form>
    
    <?php
    
    $sql = "SELECT * 
            FROM cd_categorie, Categorie_per_cd, cd
            WHERE cd.cd_ID = Categorie_per_cd.cd_ID
             AND cd_categorie.CATEGORIE_ID = Categorie_per_cd.CATEGORIE_ID
             ORDER BY naam ASC"; 
            
    $resultaat  = mysql_query($sql); // voer SQL code uit
    
    if (mysql_num_rows($resultaat) > 0) {
    echo "<p>CD overzicht<br>";
    echo "<table border=1>";    
    echo "<tr><td class=\"kolom1\"><b>Bewerk</b></td><td class=\"kolom1\"><b>titel</b></td><td class=\"kolom1\"><b>artiest</b></td><td class=\"kolom1\"><b>verschijningsjaar</b></td><td class=\"kolom1\"><b>genre</b></td></tr>"; // bovenste regel
    while ($rij = mysql_fetch_array($resultaat)) 
    {
    echo "<tr>";
    
    echo "<td><a onclick='submitcd(". $rij['CD_ID'].",\"wijzigen\")' href=\"#\" >[wijzigen]</a><br>";
    echo "<a onclick='if (confirm(\"Weet u zeker dat u de cd " . $rij['Titel'] . " wilt verwijderen?\")) submitcd(". $rij[              'CD_ID'].",\"verwijderen\")' href=\"#\" >[verwijderen]</a><br>";
    echo "</td>";
    echo "<td>".$rij["Titel"]."</td>";
    echo "<td>".$rij["Artiest"]."</td>";
    echo "<td>".$rij["Verschijningsjaar"]."</td>";
    echo "<td>".$rij["Naam"]."</td>";
    
    echo "</tr>";
    }
    echo "</table>";
    }
    echo "<p><a href=\"cd_zoek.php\">Zoek een cd via een zoekwoord of categorie</a>";
    ?>
    </div>

<?php
    
    include ('html_footer.php');
    include ('php_footer.php');
    
?>

En CD_VERWIJDEREN

PHP:

<?php

    include ('php_header.php');
//  include ('beveiliging.php');

    
        $query = "DELETE FROM cd WHERE CD_ID = ". $_POST['CD_ID'] ;
        $resultaat = mysql_query ( $query );
        $query = "DELETE FROM Categorie_per_cd WHERE CD_ID = ". $_POST['CD_ID'] ;
        $resultaat = mysql_query ( $query );

        header ('Location: admin_cd.php');
?>



<?php
    
        include ('php_footer.php');
        
?>

Ik kan geen fout ontdekken waar het hem in zit.

Ik hoop dat jullie me kunnen helpen
Bedankt!

[japaveh]

Ik zou zeggen: probeer eens de standaard debugging zoals het printen van query's etc. En oh, alsof het nog niet genoeg gezegd is: let ook goed op SQL injection...

[ChessSpider]

Zet
error_reporting(E_ALL)
bovenaan het script, en suc6 met debuggen

[link0007]

PHP:

 $query = "DELETE FROM cd WHERE CD_ID = ". $_POST['CD_ID'] ;

Wat nu als ik bij die post-data dit invul:

code:

1 OR 1=1--

?

[Verwijderd]

CD_ID komt meerdere malen voor in je query door de * en daardoor krijgen deze waarden een andere naam dan CD_ID.

[NMe]

Sorry, maar het is niet de bedoeling dat je hier een lap code post en vervolgens verwacht dat wij voor je gaan debuggen. Dat mag je lekker zelf doen. Hoe dat moet? Programming FAQ - Debuggen: hoe doe ik dat?

We willen je best helpen maar dan zul je eerst zelf eens gericht moeten gaan zoeken waar de fout ongeveer zit en alleen relevante code moeten posten. Dit is in elk geval niet de bedoeling en daarom gaat je topic ook dicht.