Hola climax2006.
Siempre inserta porque no tienes ninguna condición que diga que si hay algún error no inserte, es decir, según tu código tienes dos condiciones de error que lo único que hacen es concatenar un cadena de caracteres a la variable $msg (no muestran nada por pantalla...). Entonces da igual que sean verdaderas o falsas; el código siempre llegará a las dos últimas líneas de la condición while.
Deberías utilizar, por ejemplo, una variable para poder manejar esto, algo así:
Código PHP:
...
while (...)
{
$error = false
...
if (...) {
...
$error = true;
}
...
if (...) {
...
$error = true;
}
if (!$error) {
$insertar = "INSERT INTO..."
....
}
}
...