filterAdvisory: match installed_solvables sort with lower_bound (RhBu…

…g:2212838)

`std::lower_bound` expects that the range it operates on is sorted by
the provided comparator.

`lower_bound()` is used on `installed_solvables` twice, first with
comparator `NameSolvableComparator` and later with
`SolvableCompareAdvisoryPkgNameArch` to cover both we need to sort
`installed_solvables` by name and arch.

Otherwise this can lead to problems if multiple architectures of a pkg
are installed.

For: https://bugzilla.redhat.com/show_bug.cgi?id=2212838

libdnf/sack/query.cpp

@@ -1903,7 +1903,7 @@ Query::Impl::filterAdvisory(const Filter & f, Map *m, int keyname)
             while ((installed_id = installed.pImpl->result->next(installed_id)) != -1) {
                 installed_solvables.push_back(pool_id2solvable(pool, installed_id));
             }
-            std::sort(installed_solvables.begin(), installed_solvables.end(), NameSolvableComparator);
+            std::sort(installed_solvables.begin(), installed_solvables.end(), NameArchSolvableComparator);
 
             Query obsoletes(sack, ExcludeFlags::IGNORE_EXCLUDES);
             obsoletes.addFilter(HY_PKG, HY_EQ, resultPset);