You may also want to add a "sanity check" to the allocation of values to the BORDER and BGCOLOR lines too. Maybe even surround the values by double quotes, just in case something wierd gets passed to them which may break the HTML.