Bug 4761 (DeepinDEPackageReview) - Tracking: Deepin Desktop related package review tracker
Summary: Tracking: Deepin Desktop related package review tracker
Status: RESOLVED FIXED
Alias: DeepinDEPackageReview
Product: Package Reviews
Classification: Unclassified
Component: Review Request (show other bugs)
Version: Current
Hardware: x86_64 GNU/Linux
: P1 enhancement
Assignee: Zamir SUN
URL:
Depends on: deepin-movie deepin-music deepin-voice-recorder deepin-screen-recorder
Blocks:
  Show dependency treegraph
 
Reported: 2018-01-12 17:30 CET by Zamir SUN
Modified: 2021-10-28 20:54 CEST (History)
4 users (show)

See Also:
namespace: free


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Zamir SUN 2018-01-12 17:30:38 CET
This is a tracker for tracking Deepin Desktop related package review request.
See also https://bugzilla.redhat.com/show_bug.cgi?id=1465889 for Fedora.
Comment 1 Nicolas Chauvet 2018-09-21 16:46:20 CEST
@Zamir
Does the project still hold or is there any issue?

Please try answer the questions on each bugs
Thx
Comment 2 Zamir SUN 2018-09-24 14:54:33 CEST
Hi,

Sorry for being quiet for long. During the past months we are fixing the series of packages (dependencies) in Fedora. And I will follow up on the on-going reviews after we have fixed the packages in Fedora. And I am bit busy recently.

Another heads-up is, we decide not to package deepin-manual due to technical reasons. So some of the four reviews will be solved soon after the dependencies fixed.
Comment 3 Jeffkonaa 2021-10-28 20:54:15 CEST
This patch fixes a segfault seen when attaching to a process on Solaris.
    The steps leading to the segfault are:
    http://www.compilatori.com/tech/xiaomi/
     - procfs_target::attach calls do_attach, at this point the inferior's
       process slot in the target stack is empty. 
     - do_attach adds a thread with `add http://www.acpirateradio.co.uk/tech/forest-fires/ _thread (&the_procfs_target, ptid)`
     - in add_thread_silent, the passed target (&the_procfs_target) is
       passed to find_inferior_ptid http://www.logoarts.co.uk/tech/drone-cameras/
     - find_inferior_ptid returns nullptr, as there is no inferior with this
       ptid that has &the_procfs_target as its process target http://www.slipstone.co.uk/tech/express-van/
     - the nullptr `inf` is passed to find_thread_ptid, which dereferences
       it, causing a segfault
     - back in procfs_target::attach, after do_attach, we push the http://embermanchester.uk/technology/telegram/ 
       the_procfs_target on the inferior's target stack, although we never
       reach this because the segfault happens before. http://connstr.net/tech/mars-surface/
    
    To fix this, I think we need to do the same as is done in 
    inf_ptrace_target::attach: push the target early and unpush it in case
    the attach fails (and keep it if the attach succeeds). http://joerg.li/tech/cars-comparison/
    
    Implement it by moving target_unpush_up to target.h, so it can be
    re-used here.  Make procfs_target::attach use it.  Note that just like
    is mentioned http://www.jopspeech.com/tech/xiaomi-headset/ in inf_ptrace_target::attach, we should push the target
    before calling target_pid_to_str, so that calling target_pid_to_str ends
    up in procfs_target::pid_to_str. http://www.wearelondonmade.com/tech/driving-assistant/
    
    Tested by trying to attach on a process on gcc211 on the gcc compile
    farm.
    https://waytowhatsnext.com/health/vaccination-rates/
    gdb/ChangeLog:
This patch fixes a segfault seen when attaching to a process on Solaris.
    The steps leading to the segfault are: http://www.iu-bloomington.com/health/amoled-display/
    
     - procfs_target::attach calls do_attach, at this point the inferior's
       process slot in the target stack is empty. https://komiya-dental.com/health/telegram-account/
     - do_attach adds a thread with `add_thread (&the_procfs_target, ptid)`
     - in add_thread_silent, the passed target (&the_procfs_target) is
       passed to find_inferior_ptid http://www-look-4.com/services/usb-type-a/
     - find_inferior_ptid returns nullptr, as there is no inferior with this
       ptid that has &the_procfs_target as its process target
     - the nullptr `inf` is passed to find_thread_ptid, which dereferences
       it, causing a segfault https://www.webb-dev.co.uk/property/house-sales-in-2020/
     - back in procfs_target::attach, after do_attach, we push the
       the_procfs_target on the inferior's target stack, although we never
       reach this because the segfault happens before.
    
    To fix this, I think we need to do the same as is done in
    inf_ptrace_target::attach: push the target early and unpush it in case
    the attach fails (and keep it if the attach succeeds).