Summary: | Partition Manager crashes when run without root | ||
---|---|---|---|
Product: | [Applications] partitionmanager | Reporter: | Erin Yuki Schlarb <erin-kde> |
Component: | general | Assignee: | Andrius Štikonas <andrius> |
Status: | RESOLVED DUPLICATE | ||
Severity: | crash | Keywords: | drkonqi |
Priority: | NOR | ||
Version: | unspecified | ||
Target Milestone: | --- | ||
Platform: | Debian stable | ||
OS: | Linux | ||
Latest Commit: | Version Fixed In: |
Description
Erin Yuki Schlarb
2017-02-24 21:03:02 UTC
Yeah, known issue... Happens if you use LVM... *** This bug has been marked as a duplicate of bug 375394 *** No problem! Thanks for the reply! I can't remember ever setting up an LVM-based partition layout through… Or is it the LVM-integration module/code that causes this? (In reply to Alexander Schlarb from comment #2) > No problem! Thanks for the reply! > > I can't remember ever setting up an LVM-based partition layout through… Or > is it the LVM-integration module/code that causes this? I'm actually not sure. It's just that backtrace contains FS::lvm2_pv::getPVs, so crash must happen in that function. That function in kpmcore/src/fs/lvm2_pv.cpp loops over QList<Device*>& devices. I guess it crashes if there are no devices. Feell free to play around yourself if you have time :), should be quite easy to fix, probably one "if". If not I'll look at it at some point later, but these days I'm quite busy... Writing my PhD thesis. Thanks for the hint, I wouldn't have taken up this little bug squashing exercise otherwise! :-) As you expected there are only some very minor changes required, they just happen to be in the `libparted` backend, not in the LVM code itself. I like how easy it is to follow the structure of your code btw! Not gonna make me friends with C/C++ but it definitely makes things a lot more pleasant this way! Anyway, the code for review is here: https://git.reviewboard.kde.org/r/129968/ Have a nice weekend and thanks for KPM! It has become such an awesome tool to use! Thanks, a lot! I'll commit it... |