diff --git a/components/impl/PresentationImpl.cpp b/components/impl/PresentationImpl.cpp
--- a/components/impl/PresentationImpl.cpp
+++ b/components/impl/PresentationImpl.cpp
@@ -77,7 +77,8 @@
         foreach(const KoShape* shape, koPaView->activePage()->shapes()) {
             if(!shape->hyperLink().isEmpty()) {
                 QRectF rect = shape->boundingRect();
-                while(KoShapeContainer* parent = shape->parent()) {
+                for (KoShapeContainer* parent = shape->parent();
+                     parent; parent = parent->parent()) {
                     rect.translate(parent->position());
                 }
                 links.append(QPair<QRectF, QUrl>(rect, QUrl(shape->hyperLink())));
@@ -106,7 +107,8 @@
                                 if(shapeData->document() == text)
                                 {
                                     rect.translate(shape->position());
-                                    while(KoShapeContainer* parent = shape->parent()) {
+                                    for (KoShapeContainer* parent = shape->parent();
+                                         parent; parent = parent->parent()) {
                                         rect.translate(parent->position());
                                     }
                                     break;
diff --git a/components/impl/SpreadsheetImpl.cpp b/components/impl/SpreadsheetImpl.cpp
--- a/components/impl/SpreadsheetImpl.cpp
+++ b/components/impl/SpreadsheetImpl.cpp
@@ -79,7 +79,8 @@
         foreach(const KoShape* shape, canvas->activeSheet()->shapes()) {
             if(!shape->hyperLink().isEmpty()) {
                 QRectF rect = shape->boundingRect();
-                while(KoShapeContainer* parent = shape->parent()) {
+                for (KoShapeContainer* parent = shape->parent();
+                     parent; parent = parent->parent()) {
                     rect.translate(parent->position());
                 }
                 links.append(QPair<QRectF, QUrl>(rect, QUrl(shape->hyperLink())));
@@ -108,7 +109,8 @@
                                 if(shapeData->document() == text)
                                 {
                                     rect.translate(shape->position());
-                                    while(KoShapeContainer* parent = shape->parent()) {
+                                    for (KoShapeContainer* parent = shape->parent();
+                                         parent; parent = parent->parent()) {
                                         rect.translate(parent->position());
                                     }
                                     break;
diff --git a/components/impl/TextDocumentImpl.cpp b/components/impl/TextDocumentImpl.cpp
--- a/components/impl/TextDocumentImpl.cpp
+++ b/components/impl/TextDocumentImpl.cpp
@@ -86,7 +86,8 @@
         foreach(const KoShape* shape, canvas->shapeManager()->shapes()) {
             if(!shape->hyperLink().isEmpty()) {
                 QRectF rect = shape->boundingRect();
-                while(KoShapeContainer* parent = shape->parent()) {
+                for (KoShapeContainer* parent = shape->parent();
+                     parent; parent = parent->parent()) {
                     rect.translate(parent->position());
                 }
                 links.append(QPair<QRectF, QUrl>(rect, QUrl(shape->hyperLink())));
@@ -115,7 +116,8 @@
                                 if(shapeData->document() == text)
                                 {
                                     rect.translate(shape->position());
-                                    while(KoShapeContainer* parent = shape->parent()) {
+                                    for (KoShapeContainer* parent = shape->parent();
+                                         parent; parent = parent->parent()) {
                                         rect.translate(parent->position());
                                     }
                                     break;