Qt: Prevent loop in renames of profiles

This prevents the case, where there are two profiles, and one is
renamed, after which the second one is renamed to the firsts original
name. This could lead to undefined behavior in the underlying data
structure.

Fixes an issue as well, where deleting the identical named duplicate
would delete the original one

Bug: 15975
Change-Id: Ia7112779f2c2ef926c3ea915852b6d7b4497f777
Reviewed-on: https://code.wireshark.org/review/34189
Petri-Dish: Roland Knall <rknall@gmail.com>
Tested-by: Petri Dish Buildbot
Reviewed-by: Roland Knall <rknall@gmail.com>

5 files changed, 233 insertions, 61 deletions

diff --git a/ui/profile.c b/ui/profile.c
index 6b16e4af4f..a6742e5320 100644
--- a/ui/profile.c
+++ b/ui/profile.c
@@ -41,7 +41,7 @@ GList * edited_profile_list(void) {
 
 static GList *
 add_profile_entry(GList *fl, const char *profilename, const char *reference, int status,
-        gboolean is_global, gboolean from_global)
+        gboolean is_global, gboolean from_global, gboolean is_import)
 {
     profile_def *profile;
 
@@ -51,6 +51,7 @@ add_profile_entry(GList *fl, const char *profilename, const char *reference, int
     profile->status = status;
     profile->is_global = is_global;
     profile->from_global = from_global;
+    profile->is_import = is_import;
     return g_list_append(fl, profile);
 }
 
@@ -164,7 +165,7 @@ gchar *apply_profile_changes(void)
         g_strstrip(profile1->name);
         if (profile1->status == PROF_STAT_NEW) {
             /* We do not create a directory for the default profile */
-            if (strcmp(profile1->name, DEFAULT_PROFILE)!=0) {
+            if (strcmp(profile1->name, DEFAULT_PROFILE)!=0  && ! profile1->is_import) {
                 if (create_persconffile_profile(profile1->name, &pf_dir_path) == -1) {
                     simple_dialog(ESD_TYPE_ERROR, ESD_BTN_OK,
                             "Can't create directory\n\"%s\":\n%s.",
@@ -176,6 +177,12 @@ gchar *apply_profile_changes(void)
 
                 g_free (profile1->reference);
                 profile1->reference = g_strdup(profile1->name);
+            /* correctly apply imports as existing profiles */
+            } else if (profile1->is_import) {
+                profile1->status = PROF_STAT_EXISTS;
+                g_free (profile1->reference);
+                profile1->reference = g_strdup(profile1->name);
+                profile1->is_import = FALSE;
             }
         } else if (profile1->status == PROF_STAT_CHANGED) {
             if (strcmp(profile1->reference, profile1->name)!=0) {
@@ -235,10 +242,10 @@ gchar *apply_profile_changes(void)
 
 GList *
 add_to_profile_list(const char *name, const char *expression, int status,
-        gboolean is_global, gboolean from_global)
+        gboolean is_global, gboolean from_global, gboolean is_imported)
 {
     edited_profiles = add_profile_entry(edited_profiles, name, expression, status,
-            is_global, from_global);
+            is_global, from_global, is_imported);
 
     return g_list_last(edited_profiles);
 }
@@ -294,7 +301,7 @@ copy_profile_list(void)
 
         current_profiles = add_profile_entry(current_profiles, profile->name,
                 profile->reference, profile->status,
-                profile->is_global, profile->from_global);
+                profile->is_global, profile->from_global, FALSE);
         flp_src = g_list_next(flp_src);
     }
 }
@@ -313,7 +320,7 @@ init_profile_list(void)
     empty_profile_list(TRUE);
 
     /* Default entry */
-    add_to_profile_list(DEFAULT_PROFILE, DEFAULT_PROFILE, PROF_STAT_DEFAULT, FALSE, FALSE);
+    add_to_profile_list(DEFAULT_PROFILE, DEFAULT_PROFILE, PROF_STAT_DEFAULT, FALSE, FALSE, FALSE);
 
     /* Local (user) profiles */
     profiles_dir = get_profiles_dir();
@@ -334,7 +341,7 @@ init_profile_list(void)
     local_profiles = g_list_sort(local_profiles, (GCompareFunc)g_ascii_strcasecmp);
     for (iter = g_list_first(local_profiles); iter; iter = g_list_next(iter)) {
         name = (gchar *)iter->data;
-        add_to_profile_list(name, name, PROF_STAT_EXISTS, FALSE, FALSE);
+        add_to_profile_list(name, name, PROF_STAT_EXISTS, FALSE, FALSE, FALSE);
     }
     g_list_free_full(local_profiles, g_free);
 
@@ -357,7 +364,7 @@ init_profile_list(void)
     global_profiles = g_list_sort(global_profiles, (GCompareFunc)g_ascii_strcasecmp);
     for (iter = g_list_first(global_profiles); iter; iter = g_list_next(iter)) {
         name = (gchar *)iter->data;
-        add_to_profile_list(name, name, PROF_STAT_EXISTS, TRUE, TRUE);
+        add_to_profile_list(name, name, PROF_STAT_EXISTS, TRUE, TRUE, FALSE);
     }
     g_list_free_full(global_profiles, g_free);
 
diff --git a/ui/profile.h b/ui/profile.h
index 780bf246f6..5b6bd8d743 100644
--- a/ui/profile.h
+++ b/ui/profile.h
@@ -34,6 +34,7 @@ typedef struct {
     int       status;
     gboolean  is_global;
     gboolean  from_global;
+    gboolean  is_import;
 } profile_def;
 
 /** @file
@@ -52,11 +53,12 @@ void init_profile_list(void);
  * @param status Current status
  * @param is_global Profile is in the global configuration directory
  * @param from_global Profile is copied from the global configuration directory
+ * @param is_import Profile has been imported and no directory has to be created
  *
  * @return A pointer to the new profile list
  */
 GList *add_to_profile_list(const char *name, const char *parent, int status,
-                           gboolean is_global, gboolean from_global);
+                           gboolean is_global, gboolean from_global, gboolean is_import);
 
 /** Refresh the current (non-edited) profile list.
  */
diff --git a/ui/qt/models/profile_model.cpp b/ui/qt/models/profile_model.cpp
index 6a8f43fd43..2202e1d17f 100644
--- a/ui/qt/models/profile_model.cpp
+++ b/ui/qt/models/profile_model.cpp
@@ -164,14 +164,15 @@ void ProfileModel::loadProfiles()
 GList * ProfileModel::entry(profile_def *ref) const
 {
     GList *fl_entry = edited_profile_list();
-    while (fl_entry && fl_entry->data) {
+    while (fl_entry && fl_entry->data)
+    {
         profile_def *profile = reinterpret_cast<profile_def *>(fl_entry->data);
-        if (strcmp(ref->name, profile->name) == 0 && ref->is_global == profile->is_global)
+        if ( QString(ref->name).compare(profile->name) == 0 &&
+             QString(ref->reference).compare(profile->reference) == 0 &&
+             ref->is_global == profile->is_global &&
+             ref->status == profile->status )
         {
-            if ( ( ref->reference == Q_NULLPTR && profile->reference == Q_NULLPTR )
-                 || ( ( ref->reference != Q_NULLPTR && profile->reference != Q_NULLPTR )
-                      && (strcmp(ref->reference, profile->reference) == 0) ) )
-                return fl_entry;
+            return fl_entry;
         }
 
         fl_entry = gxx_list_next(fl_entry);
@@ -305,6 +306,105 @@ QVariant ProfileModel::dataFontRole(const QModelIndex &index) const
     return font;
 }
 
+bool ProfileModel::checkIfDeleted(int row) const
+{
+    QModelIndex idx = index(row, ProfileModel::COL_NAME);
+    return checkIfDeleted(idx);
+}
+
+bool ProfileModel::checkIfDeleted(const QModelIndex &index) const
+{
+    profile_def * prof = guard(index);
+    if ( ! prof )
+        return false;
+
+    QStringList deletedNames;
+
+    GList * current = current_profile_list();
+
+    /* search the current list as long as we have not found anything */
+    while ( current )
+    {
+        bool found = false;
+        GList * edited = edited_profile_list();
+        profile_def * profcurr = static_cast<profile_def *>(current->data);
+
+        if ( ! profcurr->is_global && profcurr->status != PROF_STAT_DEFAULT )
+        {
+            while ( edited && ! found )
+            {
+                profile_def * profed = static_cast<profile_def *>(edited->data);
+                if ( ! profed->is_global && profed->status != PROF_STAT_DEFAULT )
+                {
+                    if ( g_strcmp0(profcurr->name, profed->name) == 0 || g_strcmp0(profcurr->name, profed->reference) == 0 )
+                    {
+                        if ( profed->status == profcurr->status && prof->status != PROF_STAT_NEW && prof->status != PROF_STAT_COPY )
+                            found = true;
+                    }
+                }
+
+                edited = gxx_list_next(edited);
+            }
+
+            /* profile has been deleted, check if it has the name we ask for */
+            if ( ! found )
+                deletedNames << profcurr->name;
+        }
+
+        current = gxx_list_next(current);
+    }
+
+    if ( deletedNames.contains(prof->name) )
+        return true;
+
+    return false;
+}
+
+bool ProfileModel::checkInvalid(const QModelIndex &index) const
+{
+    profile_def * prof = guard(index);
+    if ( ! prof )
+        return false;
+
+    int ref = const_cast<ProfileModel *>(this)->findAsReference(prof->name);
+    if ( ref == index.row() )
+        return false;
+
+    profile_def * pg = guard(ref);
+    if ( pg && pg->status == PROF_STAT_CHANGED && g_strcmp0(pg->name, pg->reference) != 0 )
+        return true;
+
+    return false;
+}
+
+bool ProfileModel::checkDuplicate(const QModelIndex &index, bool isOriginalToDuplicate) const
+{
+    profile_def * prof = guard(index);
+    if ( ! prof || ( ! isOriginalToDuplicate && prof->status == PROF_STAT_EXISTS ) )
+        return false;
+
+    QList<int> rows = const_cast<ProfileModel *>(this)->findAllByNameAndVisibility(prof->name, prof->is_global, false);
+    int found = 0;
+    profile_def * check = Q_NULLPTR;
+    for(int cnt = 0; cnt < rows.count(); cnt++)
+    {
+        int row = rows.at(cnt);
+
+        if ( row == index.row() )
+            continue;
+
+        check = guard(row);
+        if ( ! check || ( isOriginalToDuplicate && check->status == PROF_STAT_EXISTS ) )
+            continue;
+
+        found++;
+    }
+
+    if ( found > 0 )
+        return true;
+    return false;
+}
+
 QVariant ProfileModel::dataBackgroundRole(const QModelIndex &index) const
 {
     if ( ! index.isValid() || profiles_.count() <= index.row() )
@@ -314,15 +414,19 @@ QVariant ProfileModel::dataBackgroundRole(const QModelIndex &index) const
     if ( ! prof )
         return QVariant();
 
-    if ( ! ProfileModel::checkNameValidity(QString(prof->name)) )
-        return ColorUtils::fromColorT(&prefs.gui_text_invalid);
-
     if ( prof->status == PROF_STAT_DEFAULT && reset_default_ )
         return ColorUtils::fromColorT(&prefs.gui_text_deprecated);
 
-    QList<int> rows = const_cast<ProfileModel *>(this)->findAllByNameAndVisibility(QString(prof->name), prof->is_global);
-    if ( rows.count() > 1 )
-        return ColorUtils::fromColorT(&prefs.gui_text_invalid);
+    if ( prof->status != PROF_STAT_DEFAULT && ! prof->is_global )
+    {
+        /* Highlights errorneous line */
+        if ( checkInvalid(index) || checkIfDeleted(index) || checkDuplicate(index) || ! checkNameValidity(prof->name) )
+            return ColorUtils::fromColorT(&prefs.gui_text_invalid);
+
+        /* Highlights line, which has been duplicated by another index */
+        if ( checkDuplicate(index, true) )
+            return ColorUtils::fromColorT(&prefs.gui_text_valid);
+    }
 
     return QVariant();
 }
@@ -337,7 +441,7 @@ QVariant ProfileModel::dataToolTipRole(const QModelIndex &idx) const
         return QVariant();
 
     if (prof->is_global)
-        return tr("This is a system provided profile.");
+        return tr("This is a system provided profile");
     else
         return dataPath(idx);
 }
@@ -351,6 +455,32 @@ QVariant ProfileModel::dataPath(const QModelIndex &index) const
     if ( ! prof )
         return QVariant();
 
+    if ( checkInvalid(index) )
+    {
+        int ref = const_cast<ProfileModel *>(this)->findAsReference(prof->name);
+        if ( ref != index.row() && ref >= 0 )
+        {
+            profile_def * prof = guard(ref);
+            QString msg = tr("A profile change for this name is pending");
+            if ( prof )
+                msg.append(tr(" (See: %1)").arg(prof->name));
+            return msg;
+        }
+
+        return tr("This is an invalid profile definition");
+    }
+
+    if ( ( prof->status == PROF_STAT_NEW || prof->status == PROF_STAT_CHANGED || prof->status == PROF_STAT_COPY ) && checkDuplicate(index) )
+        return tr("A profile already exists with this name");
+
+    if ( checkIfDeleted(index) )
+    {
+        return tr("A profile with this name is being deleted");
+    }
+
+    if ( prof->is_import )
+        return tr("Imported profile");
+
     switch (prof->status)
     {
     case PROF_STAT_DEFAULT:
@@ -358,8 +488,6 @@ QVariant ProfileModel::dataPath(const QModelIndex &index) const
             return gchar_free_to_qstring(get_persconffile_path("", FALSE));
         else
             return tr("Resetting to default");
-    case PROF_STAT_IMPORT:
-        return tr("Imported profile");
     case PROF_STAT_EXISTS:
         {
             QString profile_path;
@@ -373,12 +501,9 @@ QVariant ProfileModel::dataPath(const QModelIndex &index) const
         }
     case PROF_STAT_NEW:
         {
-            QList<int> entries = const_cast<ProfileModel *>(this)->findAllByNameAndVisibility(prof->name);
             QString errMsg;
 
-            if ( entries.count() > 1 )
-                return tr("A profile already exists with this name.");
-            else if ( ! checkNameValidity(prof->name, &errMsg) )
+            if ( ! checkNameValidity(prof->name, &errMsg) )
                 return errMsg;
             else
                 return tr("Created from default settings");
@@ -410,23 +535,25 @@ QVariant ProfileModel::dataPath(const QModelIndex &index) const
                 /* A default model as reference can neither be deleted or renamed, so skip if the reference was one */
                 else  if ( ! index.data(ProfileModel::DATA_IS_DEFAULT).toBool() )
                 {
-                    /* find all non-global, non-default profiles which are referenced by this one. Those are the only
+                    /* find a non-global, non-default profile which could be referenced by this one. Those are the only
                      * ones which could be renamed or deleted */
                     int row = const_cast<ProfileModel *>(this)->findByNameAndVisibility(prof->reference, false, true);
-                    if ( row > 0 && row != index.row() )
+                    profile_def * ref = guard(row);
+
+                    /* The reference is itself a copy of the original, therefore it is not accepted */
+                    if ( ref && ( ref->status == PROF_STAT_COPY || ref->status == PROF_STAT_NEW ) && QString(ref->name).compare(prof->reference) != 0 )
+                        ref = Q_NULLPTR;
+
+                    /* found no other profile, original one had to be deleted */
+                    if ( ! ref || row == index.row() || checkIfDeleted(row) )
                     {
-                        /* found another profile, so the reference had been renamed, it the status is changed */
-                        profile_def * ref = guard(row);
-                        if ( ref && ref->status == PROF_STAT_CHANGED )
-                            appendix = tr("renamed to %1").arg(ref->name);
+                        appendix = tr("deleted");
                     }
-                    else
+                    /* found another profile, so the reference had been renamed, it the status is changed */
+                    else if ( ref && ref->status == PROF_STAT_CHANGED )
                     {
-                        /* found no other profile, original one had to be deleted */
-                        appendix = tr("deleted");
+                        appendix = tr("renamed to %1").arg(ref->name);
                     }
-
-
                 }
 
                 if ( appendix.length() > 0 )
@@ -485,7 +612,7 @@ QVariant ProfileModel::data(const QModelIndex &index, int role) const
     case ProfileModel::DATA_PATH_IS_NOT_DESCRIPTION:
         if ( prof->status == PROF_STAT_NEW || prof->status == PROF_STAT_COPY
              || ( prof->status == PROF_STAT_DEFAULT && reset_default_ )
-             || prof->status == PROF_STAT_CHANGED || prof->status == PROF_STAT_IMPORT )
+             || prof->status == PROF_STAT_CHANGED || prof->is_import )
             return qVariantFromValue(false);
         else
             return qVariantFromValue(true);
@@ -538,6 +665,22 @@ int ProfileModel::findByName(QString name)
     return row;
 }
 
+int ProfileModel::findAsReference(QString reference)
+{
+    int found = -1;
+    if ( reference.length() <= 0 )
+        return found;
+
+    for ( int cnt = 0; cnt < profiles_.count() && found < 0; cnt++ )
+    {
+        profile_def * prof = guard(cnt);
+        if ( prof && reference.compare(prof->reference) == 0 )
+            found = cnt;
+    }
+
+    return found;
+}
+
 int ProfileModel::findByNameAndVisibility(QString name, bool isGlobal, bool searchReference)
 {
     QList<int> result = findAllByNameAndVisibility(name, isGlobal, searchReference);
@@ -572,7 +715,7 @@ QModelIndex ProfileModel::addNewProfile(QString name)
         cnt++;
     }
 
-    add_to_profile_list(newName.toUtf8().constData(), Q_NULLPTR, PROF_STAT_NEW, FALSE, FALSE);
+    add_to_profile_list(newName.toUtf8().constData(), newName.toUtf8().constData(), PROF_STAT_NEW, FALSE, FALSE, FALSE);
     loadProfiles();
 
     return index(findByName(newName), COL_NAME);
@@ -645,7 +788,7 @@ QModelIndex ProfileModel::duplicateEntry(QModelIndex idx, int new_status)
         new_status = PROF_STAT_NEW;
 
     /* add element */
-    add_to_profile_list(new_name.toUtf8().constData(), parent.toUtf8().constData(), new_status, FALSE, prof->from_global ? prof->from_global : prof->is_global);
+    add_to_profile_list(new_name.toUtf8().constData(), parent.toUtf8().constData(), new_status, FALSE, prof->from_global ? prof->from_global : prof->is_global, FALSE);
 
     /* reload profile list in model */
     loadProfiles();
@@ -755,6 +898,7 @@ bool ProfileModel::setData(const QModelIndex &idx, const QVariant &value, int ro
     if ( role != Qt::EditRole ||  ! value.isValid() || value.toString().isEmpty() )
         return false;
 
+    QString newValue = value.toString();
     profile_def * prof = guard(idx);
     if ( ! prof || prof->status == PROF_STAT_DEFAULT )
         return false;
@@ -762,12 +906,12 @@ bool ProfileModel::setData(const QModelIndex &idx, const QVariant &value, int ro
     last_set_row_ = idx.row();
 
     QString current(prof->name);
-    if ( current.compare(value.toString()) != 0 )
+    if ( current.compare(newValue) != 0 )
     {
         g_free(prof->name);
-        prof->name = qstring_strdup(value.toString());
+        prof->name = qstring_strdup(newValue);
 
-        if (prof->reference && strcmp(prof->name, prof->reference) == 0) {
+        if (prof->reference && g_strcmp0(prof->name, prof->reference) == 0 && ! ( prof->status == PROF_STAT_NEW || prof->status == PROF_STAT_COPY )) {
             prof->status = PROF_STAT_EXISTS;
         } else if (prof->status == PROF_STAT_EXISTS) {
             prof->status = PROF_STAT_CHANGED;
@@ -1003,7 +1147,10 @@ int ProfileModel::importProfilesFromDir(QString dirname, int * skippedCnt, bool
 
             success = copyTempToProfile(tempPath, profilePath, &wasEmpty);
             if ( success )
+            {
                 count++;
+                add_to_profile_list(fentry.fileName().toUtf8().constData(), fentry.fileName().toUtf8().constData(), PROF_STAT_NEW, FALSE, FALSE, TRUE);
+            }
             else if ( ! wasEmpty && QFile::exists(profilePath) )
             {
                 QDir dh(profilePath);
@@ -1038,8 +1185,7 @@ void ProfileModel::markAsImported(QStringList importedItems)
         if ( ! prof )
             continue;
 
-        prof->status = PROF_STAT_IMPORT;
-        prof->from_global = true;
+        prof->is_import = true;
     }
 }
 
@@ -1050,7 +1196,7 @@ bool ProfileModel::clearImported(QString *msg)
     for ( int cnt = 0; cnt < rowCount(); cnt++ )
     {
         profile_def * prof = guard(cnt);
-        if ( prof && prof->status == PROF_STAT_IMPORT && ! rows.contains(cnt) )
+        if ( prof && prof->is_import && ! rows.contains(cnt) )
             rows << cnt;
     }
     /* Security blanket. This ensures, that we start deleting from the end and do not get any issues iterating the list */
@@ -1064,7 +1210,7 @@ bool ProfileModel::clearImported(QString *msg)
         {
             if ( msg )
             {
-                QString errmsg = QString("%1\n\"%2\":\n%3.").arg(tr("Can't delete profile directory")).arg(ret_path).arg(g_strerror(errno));
+                QString errmsg = QString("%1\n\"%2\":\n%3").arg(tr("Can't delete profile directory")).arg(ret_path).arg(g_strerror(errno));
                 msg->append(errmsg);
             }
 
@@ -1104,7 +1250,7 @@ bool ProfileModel::checkNameValidity(QString name, QString *msg)
         message = tr("A profile cannot start or end with a period (.)");
 #else
     if ( invalid )
-        message = tr("A profile name cannot contain the '/' character.");
+        message = tr("A profile name cannot contain the '/' character");
 #endif
 
     if (! message.isEmpty()) {
diff --git a/ui/qt/models/profile_model.h b/ui/qt/models/profile_model.h
index 71ab762419..82678144f9 100644
--- a/ui/qt/models/profile_model.h
+++ b/ui/qt/models/profile_model.h
@@ -111,6 +111,11 @@ public:
 
     int lastSetRow() const;
 
+    bool checkInvalid(const QModelIndex &index) const;
+    bool checkIfDeleted(const QModelIndex &index) const;
+    bool checkIfDeleted(int row) const;
+    bool checkDuplicate(const QModelIndex &index, bool isOriginalToDuplicate = false) const;
+
 Q_SIGNALS:
     void itemChanged(const QModelIndex &idx);
 
@@ -129,6 +134,7 @@ private:
     GList * entry(profile_def *) const;
 
     int findByNameAndVisibility(QString name, bool isGlobal = false, bool searchReference = false);
+    int findAsReference(QString reference);
 
 #ifdef HAVE_MINIZIP
     static bool acceptFile(QString fileName, int fileSize);
diff --git a/ui/qt/profile_dialog.cpp b/ui/qt/profile_dialog.cpp
index 8d3d87052e..3d82068a40 100644
--- a/ui/qt/profile_dialog.cpp
+++ b/ui/qt/profile_dialog.cpp
@@ -240,12 +240,12 @@ void ProfileDialog::updateWidgets()
     if ( model_->changesPending() )
     {
         enable_import = false;
-        msg = tr("An import of profiles is not allowed, while changes are pending.");
+        msg = tr("An import of profiles is not allowed, while changes are pending");
     }
     else if ( model_->importPending() )
     {
         enable_import = false;
-        msg = tr("An import is pending to be saved. Additional imports are not allowed.");
+        msg = tr("An import is pending to be saved. Additional imports are not allowed");
     }
     import_button_->setToolTip( msg );
     import_button_->setEnabled( enable_import );
@@ -265,9 +265,9 @@ void ProfileDialog::updateWidgets()
     if ( ! enable_export )
     {
         if ( ! contains_user )
-            export_button_->setToolTip(tr("An export of profiles is only allowed for personal profiles."));
+            export_button_->setToolTip(tr("An export of profiles is only allowed for personal profiles"));
         else
-            export_button_->setToolTip(tr("An export of profiles is not allowed, while changes are pending."));
+            export_button_->setToolTip(tr("An export of profiles is not allowed, while changes are pending"));
     }
     export_selected_entry_->setVisible(contains_user);
 #endif
@@ -286,6 +286,7 @@ void ProfileDialog::updateWidgets()
             enable_del = false;
     }
 
+    QString hintUrl;
     msg.clear();
     if ( multiple )
     {
@@ -294,7 +295,6 @@ void ProfileDialog::updateWidgets()
 
         msg = tr("%Ln selected personal profile(s)", "", user_profiles);
         pd_ui_->hintLabel->setText(msg);
-        pd_ui_->hintLabel->setUrl(QString());
 #ifdef HAVE_MINIZIP
         export_selected_entry_->setText(msg);
 #endif
@@ -305,10 +305,8 @@ void ProfileDialog::updateWidgets()
         if ( index.isValid() )
         {
             QString temp = index.data(ProfileModel::DATA_PATH).toString();
-            if ( index.data(ProfileModel::DATA_PATH_IS_NOT_DESCRIPTION).toBool() )
-                pd_ui_->hintLabel->setUrl(QUrl::fromLocalFile(temp).toString());
-            else
-                pd_ui_->hintLabel->setUrl(QString());
+            if ( index.data(ProfileModel::DATA_PATH_IS_NOT_DESCRIPTION).toBool() && QFileInfo(temp).isDir() )
+                hintUrl = QUrl::fromLocalFile(temp).toString();
             pd_ui_->hintLabel->setText(temp);
             pd_ui_->hintLabel->setToolTip(index.data(Qt::ToolTipRole).toString());
 
@@ -333,13 +331,24 @@ void ProfileDialog::updateWidgets()
 
             if ( ! ProfileModel::checkNameValidity(name, &msg) )
             {
-                pd_ui_->hintLabel->setText(msg);
-                pd_ui_->hintLabel->setUrl(QString());
+                if ( idx == index || selectedProfiles().contains(idx) )
+                {
+                    hintUrl.clear();
+                    pd_ui_->hintLabel->setText(msg);
+                }
 
                 enable_ok = false;
                 continue;
             }
 
+            if ( model_->checkInvalid(idx) || model_->checkIfDeleted(idx) )
+            {
+                if ( idx == index )
+                    hintUrl.clear();
+                enable_ok = false;
+                continue;
+            }
+
             if ( idx != index && idx.data().toString().compare(index.data().toString()) == 0 )
             {
                 if (idx.data(ProfileModel::DATA_IS_GLOBAL).toBool() == index.data(ProfileModel::DATA_IS_GLOBAL).toBool())
@@ -352,6 +361,8 @@ void ProfileDialog::updateWidgets()
         }
     }
 
+    pd_ui_->hintLabel->setUrl(hintUrl);
+
     /* ensure the name column is resized to it's content */
     pd_ui_->profileTreeView->resizeColumnToContents(ProfileModel::COL_NAME);