Bug 1061911
| Summary: | [abrt] NetworkManager-0.9.9.0-20.git20131003.fc20: _g_log_abort: Process /usr/sbin/NetworkManager was killed by signal 6 (SIGABRT) | ||||||||||||||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
| Product: | [Fedora] Fedora | Reporter: | Javier Orlando Ramirez Martinez <javier.ramirez> | ||||||||||||||||||||||
| Component: | NetworkManager | Assignee: | Jirka Klimes <jklimes> | ||||||||||||||||||||||
| Status: | CLOSED ERRATA | QA Contact: | Fedora Extras Quality Assurance <extras-qa> | ||||||||||||||||||||||
| Severity: | unspecified | Docs Contact: | |||||||||||||||||||||||
| Priority: | unspecified | ||||||||||||||||||||||||
| Version: | 20 | CC: | danw, dcbw, jklimes, rkhan | ||||||||||||||||||||||
| Target Milestone: | --- | ||||||||||||||||||||||||
| Target Release: | --- | ||||||||||||||||||||||||
| Hardware: | x86_64 | ||||||||||||||||||||||||
| OS: | Unspecified | ||||||||||||||||||||||||
| URL: | https://retrace.fedoraproject.org/faf/reports/bthash/47fe6387bc1e884429b7a14a50ec4d08ac3011f5 | ||||||||||||||||||||||||
| Whiteboard: | abrt_hash:ec58b7b53ebccaa391ebd1ebd4c3f9015556d5f2 | ||||||||||||||||||||||||
| Fixed In Version: | NetworkManager-0.9.9.0-38.git20131003.fc20 | Doc Type: | Bug Fix | ||||||||||||||||||||||
| Doc Text: | Story Points: | --- | |||||||||||||||||||||||
| Clone Of: | Environment: | ||||||||||||||||||||||||
| Last Closed: | 2014-04-22 03:59:34 UTC | Type: | --- | ||||||||||||||||||||||
| Regression: | --- | Mount Type: | --- | ||||||||||||||||||||||
| Documentation: | --- | CRM: | |||||||||||||||||||||||
| Verified Versions: | Category: | --- | |||||||||||||||||||||||
| oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |||||||||||||||||||||||
| Cloudforms Team: | --- | Target Upstream Version: | |||||||||||||||||||||||
| Embargoed: | |||||||||||||||||||||||||
| Attachments: |
|
||||||||||||||||||||||||
|
Description
Javier Orlando Ramirez Martinez
2014-02-05 21:09:50 UTC
Created attachment 859871 [details]
File: backtrace
Created attachment 859872 [details]
File: cgroup
Created attachment 859873 [details]
File: core_backtrace
Created attachment 859874 [details]
File: dso_list
Created attachment 859875 [details]
File: environ
Created attachment 859876 [details]
File: limits
Created attachment 859877 [details]
File: maps
Created attachment 859878 [details]
File: open_fds
Created attachment 859879 [details]
File: proc_pid_status
Created attachment 859880 [details]
File: var_log_messages
Please consider updating NetworkManager. There were some changes in the code. Nevertheless, there is still an issue upstream. jk/secret-agent-fix branch contains a fix. the change seems generally correct, but can you explain how the current code is causing problems? Agreed that the change seems correct, and may actually "fix" the bug but not the actual root cause. --- So the actual crash here appears to be due to agent_dbus_owner == NULL, and thus the creation of the auth chain fails because NM can't create an auth chain for the agent in get_next_cb(). So the question is why agent_dbus_owner is NULL... agent_dbus_owner comes from nm_secret_agent_get_dbus_owner(), and when creating the agent, the owner must be valid. The only place the owner is freed is nm-secret-agent.c in dispose(). So for agent_dbus_owner to be NULL, the current agent must have been already disposed. For the current agent to already be disposed, it must have had remove_agent() called on it, since (barring refcount problems) that's the only place that agents are disposed of. But that's really odd, because the code takes pains to ensure that when an agent is disposed, it gets removed from req->pending, and can never become the current agent. And if it's already the current agent, then it should get dropped from the request, and the next agent asked. ---- I did see one thing though. get_done_cb() uses: g_return_if_fail (call_id == parent->current_call_id); to ensure that a response from some agent that we don't care about anymore is ignored. Well, it turns out that call_id and parent->current_call_id are the return value from dbus_g_proxy_begin_call(), which is a counter that starts at 1 and counts up for each call to that proxy object. But *each* agent (and therefore each proxy) will start counting at 1. So NM sends a request to agent A, and then drops agent A, and then sends a request to agent B, and then finally agent A replies, there's a chance that NM thinks that agent A's response is actually for agent B. I don't see how that could cause the crash above, but it's worth noting and probably worth fixing, by having NMSecretAgent pass back the pointer to 'r' (which would still be opaque) instead of r->call, to ensure uniqueness. Pushed to upstream master: 4141e69 settings: free memory in finalize(), not in dispose() in NMSecretAgent (rh #1061911) However, I leave the bug opened, so that it can be inspected deeper with regards to comment 13. NetworkManager-0.9.9.0-34.git20131003.fc20 has been submitted as an update for Fedora 20. https://admin.fedoraproject.org/updates/NetworkManager-0.9.9.0-34.git20131003.fc20 Package NetworkManager-0.9.9.0-34.git20131003.fc20: * should fix your issue, * was pushed to the Fedora 20 testing repository, * should be available at your local mirror within two days. Update it with: # su -c 'yum update --enablerepo=updates-testing NetworkManager-0.9.9.0-34.git20131003.fc20' as soon as you are able to. Please go to the following url: https://admin.fedoraproject.org/updates/FEDORA-2014-4964/NetworkManager-0.9.9.0-34.git20131003.fc20 then log in and leave karma (feedback). NetworkManager-0.9.9.0-35.git20131003.fc20 has been submitted as an update for Fedora 20. https://admin.fedoraproject.org/updates/NetworkManager-0.9.9.0-35.git20131003.fc20 NetworkManager-0.9.9.0-36.git20131003.fc20 has been submitted as an update for Fedora 20. https://admin.fedoraproject.org/updates/NetworkManager-0.9.9.0-36.git20131003.fc20 NetworkManager-0.9.9.0-37.git20131003.fc20 has been submitted as an update for Fedora 20. https://admin.fedoraproject.org/updates/NetworkManager-0.9.9.0-37.git20131003.fc20 NetworkManager-0.9.9.0-38.git20131003.fc20 has been submitted as an update for Fedora 20. https://admin.fedoraproject.org/updates/NetworkManager-0.9.9.0-38.git20131003.fc20 NetworkManager-0.9.9.0-38.git20131003.fc20 has been pushed to the Fedora 20 stable repository. If problems still persist, please make note of it in this bug report. |