Bug 487128 - Attaching to a process - errors are not being propagated to the user
Summary: Attaching to a process - errors are not being propagated to the user
Status: RESOLVED FIXED
Alias: None
Product: CDT
Classification: Tools
Component: cdt-debug-dsf-gdb (show other bugs)
Version: Next   Edit
Hardware: PC Linux
: P3 normal (vote)
Target Milestone: 9.0.0   Edit
Assignee: Project Inbox CLA
QA Contact: Marc Khouzam CLA
URL:
Whiteboard:
Keywords:
: 426308 (view as bug list)
Depends on:
Blocks:
 
Reported: 2016-02-03 09:45 EST by Marc Dumais CLA
Modified: 2016-03-03 11:32 EST (History)
4 users (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Marc Dumais CLA 2016-02-03 09:45:03 EST 
When the user connects/attaches to a process, a number of things can go wrong. However, ATM, the attaching just silently fails.

Examples: 
- attaching to a process to which the user does not have permission (owned by another user)
- attaching again to a process that one is already debugging (maybe by mistake while trying to attach to another instance of that process)
- OS does not permit attaching (e.g. on Ubuntu, by default, for unprivileged users)

It would be nice for the user to get feedback about what went wrong in these and probably other cases.
Comment 1 Marc-André Laperle CLA 2016-02-03 09:49:53 EST 
If this is fixed, this will probably fix bug 426308 as well?
Comment 2 Marc Dumais CLA 2016-02-03 12:45:19 EST 
(In reply to Marc-Andre Laperle from comment #1)
> If this is fixed, this will probably fix bug 426308 as well?

I had not noticed that bug; thanks for making me aware. Yes, I think it looks like my bug is a super-set of yours :)  Do you want to make yours broader in scope and we'll mark mine as duplicate? From what I see, I think the issue happens also when not using gdbserver - in local, non-stop, debug sessions.
Comment 3 Marc-André Laperle CLA 2016-02-03 13:23:51 EST 
*** Bug 426308 has been marked as a duplicate of this bug. ***
Comment 4 Marc-André Laperle CLA 2016-02-03 13:24:52 EST 
Let's use this bug. When there is a fix, we can make sure all use cases work (local, remote, etc).
Comment 5 Eclipse Genie CLA 2016-02-09 14:34:42 EST 
New Gerrit change created: https://git.eclipse.org/r/66238
Comment 6 Eclipse Genie CLA 2016-02-26 16:27:41 EST 
New Gerrit change created: https://git.eclipse.org/r/67462
Comment 8 Marc-André Laperle CLA 2016-02-28 12:03:07 EST 
(In reply to Eclipse Genie from comment #7)
> Gerrit change https://git.eclipse.org/r/67462 was merged to [master].
> Commit:
> http://git.eclipse.org/c/cdt/org.eclipse.cdt.git/commit/
> ?id=21e303d976c8c2c260d906784e19e3456cacabe7

Did we verify that bug 426308 was fixed as well?
Comment 9 Marc Dumais CLA 2016-02-29 06:29:34 EST 
(In reply to Marc-Andre Laperle from comment #8)
> Did we verify that bug 426308 was fixed as well?

What was committed was only a partial fix. There is still a review ongoing for the rest: 
https://git.eclipse.org/r/#/c/66238/
Comment 11 Marc Dumais CLA 2016-03-03 11:32:12 EST 
Added a note for this in the N&N.