Problemer med to get-variabler inni hverandre.

[Rinox]

Hei! Jeg har et problem nå som jeg rett og slett ikke kan forstå. Jeg har laget et script nå der jeg først av alt kan velge en side å gå til. Dette setter variabelen $_GET['d']. På hver av sidene man kan velge har jeg et lite script og det er i der problemet dukker opp. Jeg får ingen feilmelding og ingenting fra scriptet vises på skjermen, heller ikke teksten "funker dette?". Dersom jeg fjerner følgende linjer vises alt fra scriptet på skjermen (selvfølgelig utenom linjene jeg fjernet og det er jo problemet!) men jeg trenger jo de linjene...

 

	if (isset($_GET['finish'])) {
	$sql = "UPDATE todo_list SET status='2', WHERE id = '" . $_GET['finish'] . "' AND project_id = '" . $project['project_id']"'";
	mysql_query($sql);
}

if (isset($_GET['delete'])) {
	$sql = "DELETE * FROM todo_list WHERE id = '" . $_GET['delete'] . "' AND project_id = '" . $project['project_id']"'";
	mysql_query($sql);
}

 

Er det noen her som vet hvordan jeg kan fikse problemet mitt? Det burde da ikke være et så stort problem...?

 

Her er koden til scriptet som befinner seg på den valgte siden:

 

<?php
echo "funker dette?";
if (isset($_GET['d']) and $_GET['d'] != '') {

if (isset($_GET['finish'])) {
	$sql = "UPDATE todo_list SET status='2', WHERE id = '" . $_GET['finish'] . "' AND project_id = '" . $project['project_id']"'";
	mysql_query($sql);
}

if (isset($_GET['delete'])) {
	$sql = "DELETE * FROM todo_list WHERE id = '" . $_GET['delete'] . "' AND project_id = '" . $project['project_id']"'";
	mysql_query($sql);
}

$result = mysql_query("SELECT * FROM todo_list WHERE project_id='" . $project['project_id'] . "' and status='1'") or die(mysql_error());
?>
<h3>Listen</h3>
	<ul>
	<?php

	while($todo = mysql_fetch_array($result)) {
		echo '
		<li>' . $todo['text'] . '
		<a href="index.php?d=' . $project['name'] . '&finish=' . $todo['id'] . '"><img src="' . $styles_dir . '/tick.png" title="Clear" /></a>
		<a href="index.php?d=' . $project['name'] . '&delete=' . $todo['id'] . '"><img src="' . $styles_dir . '/cross.png" title="Delete" /></a>
		</li>';
	}
	?>
	</ul>

<?php
}
?>

 

Tusen takk for all hjelp!!

Endret 30. desember 2008 av Rinox

[Jonas]

Du har en syntaksfeil begge steder hvor du setter $sql. Kan også nevne at scriptet ditt er ekstremt lite sikkert og at noen når som helst kan slette hele todo_list.

[Rinox]

Tusen takk! Jeg fikk det til nå takket være deg. Jeg hadde utelatt et punktum som bindeledd mellom strengene i $sql

 

Jeg vet det er et veldig (eller ekstremt...) usikkert script (trodde jeg sa det i første post , men i og med at det kun skal kjøres av meg på mitt lokle intranett regnet jeg med at det går bra, men nå som du sier det er det vel mulig for hackere å bryte seg inn på intranettet mitt også, eller hva?

Er det viktig med sikkerhet på et script som kun skal brukes på localhost, men likevel kanskje inneholde viktig informasjon? Sier seg vel nesten selv at det er det, men hva slags angrep kan en hacker bruke? Bare slik at jeg vet hva jeg bør beskytte meg mot...

 

Jeg skal ta en real overhaling med sikkerhet på scriptet i dag uansett