> for MAGFILE in $MAGDIR/*.zip Don't forget the double quotes around variable references. It's better to always do that by default than to fix it afterwards (either because you feed it paths with whitespace in them yourself at some point or because someone else is trying to close the safety holes after you publish such a script). for MAGFILE in "$MAGDIR"/*.zip