ну... во-первых у Вас link1 и link2 это локальные переменные функции myXML.onLoad();
а в обработчиках кнопок соответственно эти переменные неизвестны (undefined),
т.е их определение нужно вынести за пределы этой функции...
например вот так:

Код:
var link1:String = "";
var link2:String = "";
news1_btn.onRelease = function():Void
{
getURL(link1, "_blank");
};
var myXML:XML = new XML();
myXML.load("p1/1.xml");
myXML.ignoreWhite = true;
myXML.onLoad = function(success):Void
{
if (success) {
trace("load was good");
var aURLs:Array = this.firstChild.childNodes;
for (var i:Number = 0; i < aURLs.length; i++) {
_root["link" + (i + 1)] = aURLs[i].attributes.url;
//trace(_root["link" + (i + 1)]);
}
} else {
trace("XML failed to load");
}
};
и при этом коде XML-файл следующей структуры:

Код:
<?xml version="1.0" encoding="UTF-8"?>
<links>
<link url = "http://mail.ru" />
<link url = "http://yandex.ru" />
</links>