Re: Review Request 129733: KSycoca: don't follow symlink to directories, it creates a risk of recursion
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/129733/ --- (Updated Jan. 3, 2017, 11:37 a.m.) Status -- This change has been marked as submitted. Review request for KDE Frameworks, Albert Astals Cid and René J.V. Bertin. Changes --- Submitted with commit 8ff849760a57b3245daef16552f55e48c8c9424d by David Faure to branch master. Repository: kservice Description --- Symlinks to (desktop) files are still supported. Diffs - autotests/ksycocatest.cpp 4564ee6750a80a109492c197c83944e129dcabac src/sycoca/ksycocautils_p.h f144a80ea6d845ed5eaff41689756b30f17d9391 src/sycoca/vfolder_menu.cpp 5acbf8ac21a1a0170cee4aa96882bd97eb2a4f2a Diff: https://git.reviewboard.kde.org/r/129733/diff/ Testing --- Added unittest (which failed before the fix, passes after the fix) Thanks, David Faure
Re: Review Request 129733: KSycoca: don't follow symlink to directories, it creates a risk of recursion
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/129733/#review101744 --- Ship it! Ship It! - René J.V. Bertin On Jan. 2, 2017, 9:57 p.m., David Faure wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/129733/ > --- > > (Updated Jan. 2, 2017, 9:57 p.m.) > > > Review request for KDE Frameworks, Albert Astals Cid and René J.V. Bertin. > > > Repository: kservice > > > Description > --- > > Symlinks to (desktop) files are still supported. > > > Diffs > - > > autotests/ksycocatest.cpp 4564ee6750a80a109492c197c83944e129dcabac > src/sycoca/ksycocautils_p.h f144a80ea6d845ed5eaff41689756b30f17d9391 > src/sycoca/vfolder_menu.cpp 5acbf8ac21a1a0170cee4aa96882bd97eb2a4f2a > > Diff: https://git.reviewboard.kde.org/r/129733/diff/ > > > Testing > --- > > Added unittest (which failed before the fix, passes after the fix) > > > Thanks, > > David Faure > >
Re: Review Request 129733: KSycoca: don't follow symlink to directories, it creates a risk of recursion
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/129733/ --- (Updated Jan. 2, 2017, 8:57 p.m.) Review request for KDE Frameworks, Albert Astals Cid and René J.V. Bertin. Changes --- Harmonize checks between the two places. Repository: kservice Description --- Symlinks to (desktop) files are still supported. Diffs (updated) - autotests/ksycocatest.cpp 4564ee6750a80a109492c197c83944e129dcabac src/sycoca/ksycocautils_p.h f144a80ea6d845ed5eaff41689756b30f17d9391 src/sycoca/vfolder_menu.cpp 5acbf8ac21a1a0170cee4aa96882bd97eb2a4f2a Diff: https://git.reviewboard.kde.org/r/129733/diff/ Testing --- Added unittest (which failed before the fix, passes after the fix) Thanks, David Faure
Re: Review Request 129733: KSycoca: don't follow symlink to directories, it creates a risk of recursion
> On Jan. 2, 2017, 8:29 p.m., René J.V. Bertin wrote: > > Ship It! > > René J.V. Bertin wrote: > Oops, I was too fast. > > David, on a whim I just tried > > ``` > sudo ln -s ../../.. /opt/local/share/applications/kde4/testlink > ``` > > which is the exact same kind of tarpit situation that got me the other > day. Your patch doesn't protect against it. Reproduced. As I suspected it's the missing check in vfolder_menu.cpp. Updated patch coming up. - David --- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/129733/#review101740 --- On Dec. 31, 2016, 1:06 p.m., David Faure wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/129733/ > --- > > (Updated Dec. 31, 2016, 1:06 p.m.) > > > Review request for KDE Frameworks, Albert Astals Cid and René J.V. Bertin. > > > Repository: kservice > > > Description > --- > > Symlinks to (desktop) files are still supported. > > > Diffs > - > > autotests/ksycocatest.cpp 4564ee6750a80a109492c197c83944e129dcabac > src/sycoca/ksycocautils_p.h f144a80ea6d845ed5eaff41689756b30f17d9391 > > Diff: https://git.reviewboard.kde.org/r/129733/diff/ > > > Testing > --- > > Added unittest (which failed before the fix, passes after the fix) > > > Thanks, > > David Faure > >
Re: Review Request 129733: KSycoca: don't follow symlink to directories, it creates a risk of recursion
> On Jan. 2, 2017, 9:29 p.m., René J.V. Bertin wrote: > > Ship It! Oops, I was too fast. David, on a whim I just tried ``` sudo ln -s ../../.. /opt/local/share/applications/kde4/testlink ``` which is the exact same kind of tarpit situation that got me the other day. Your patch doesn't protect against it. - René J.V. --- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/129733/#review101740 --- On Dec. 31, 2016, 2:06 p.m., David Faure wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/129733/ > --- > > (Updated Dec. 31, 2016, 2:06 p.m.) > > > Review request for KDE Frameworks, Albert Astals Cid and René J.V. Bertin. > > > Repository: kservice > > > Description > --- > > Symlinks to (desktop) files are still supported. > > > Diffs > - > > autotests/ksycocatest.cpp 4564ee6750a80a109492c197c83944e129dcabac > src/sycoca/ksycocautils_p.h f144a80ea6d845ed5eaff41689756b30f17d9391 > > Diff: https://git.reviewboard.kde.org/r/129733/diff/ > > > Testing > --- > > Added unittest (which failed before the fix, passes after the fix) > > > Thanks, > > David Faure > >
Re: Review Request 129733: KSycoca: don't follow symlink to directories, it creates a risk of recursion
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/129733/#review101740 --- Ship it! Ship It! - René J.V. Bertin On Dec. 31, 2016, 2:06 p.m., David Faure wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/129733/ > --- > > (Updated Dec. 31, 2016, 2:06 p.m.) > > > Review request for KDE Frameworks, Albert Astals Cid and René J.V. Bertin. > > > Repository: kservice > > > Description > --- > > Symlinks to (desktop) files are still supported. > > > Diffs > - > > autotests/ksycocatest.cpp 4564ee6750a80a109492c197c83944e129dcabac > src/sycoca/ksycocautils_p.h f144a80ea6d845ed5eaff41689756b30f17d9391 > > Diff: https://git.reviewboard.kde.org/r/129733/diff/ > > > Testing > --- > > Added unittest (which failed before the fix, passes after the fix) > > > Thanks, > > David Faure > >
Re: Review Request 129733: KSycoca: don't follow symlink to directories, it creates a risk of recursion
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/129733/#review101688 --- Ship it! Ship It! - Aleix Pol Gonzalez On Dec. 31, 2016, 2:06 p.m., David Faure wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/129733/ > --- > > (Updated Dec. 31, 2016, 2:06 p.m.) > > > Review request for KDE Frameworks, Albert Astals Cid and René J.V. Bertin. > > > Repository: kservice > > > Description > --- > > Symlinks to (desktop) files are still supported. > > > Diffs > - > > autotests/ksycocatest.cpp 4564ee6750a80a109492c197c83944e129dcabac > src/sycoca/ksycocautils_p.h f144a80ea6d845ed5eaff41689756b30f17d9391 > > Diff: https://git.reviewboard.kde.org/r/129733/diff/ > > > Testing > --- > > Added unittest (which failed before the fix, passes after the fix) > > > Thanks, > > David Faure > >
Review Request 129733: KSycoca: don't follow symlink to directories, it creates a risk of recursion
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/129733/ --- Review request for KDE Frameworks, Albert Astals Cid and René J.V. Bertin. Repository: kservice Description --- Symlinks to (desktop) files are still supported. Diffs - autotests/ksycocatest.cpp 4564ee6750a80a109492c197c83944e129dcabac src/sycoca/ksycocautils_p.h f144a80ea6d845ed5eaff41689756b30f17d9391 Diff: https://git.reviewboard.kde.org/r/129733/diff/ Testing --- Added unittest (which failed before the fix, passes after the fix) Thanks, David Faure