Ticket #1274 (closed defect: fixed)

Opened 2 years ago

Last modified 2 years ago

Wrong resolv_status might cause crash when name resolution times out

Reported by: jhrozek Owned by: jhrozek
Priority: major Milestone: SSSD 1.8.2 (LTM)
Component: Async Resolver Version: 1.8.1
Keywords: Cc:
Blocked By: Blocking:
Tests Updated: no Coverity Bug:
Patch Submitted: yes Red Hat Bugzilla: 805918, 812387
Design link:
Feature Milestone:
Design review: Fedora test page:
Chosen: Candidate to push out:
Release Notes:

Description

When name resolution times out, the errno return code is 110 (which is correct = ETIMEDOUT), but the resolv_status is 0. This is wrong.

We should probably rely mostly on the errno return code and only treat the resolv_status, which is the original ares return code, as kind of extended information, also because in the future we might switch to a different resolver with different error codes.

Change History

comment:1 Changed 2 years ago by dpal

  • Milestone changed from NEEDS_TRIAGE to SSSD 1.8.2 (LTM)

comment:2 Changed 2 years ago by dpal

  • Red Hat Bugzilla set to [https://bugzilla.redhat.com/show_bug.cgi?id=805918 805918]

Ticket has been cloned to Bugzilla: https://bugzilla.redhat.com/show_bug.cgi?id=805918

comment:3 Changed 2 years ago by jhrozek

  • Patch Submitted set
  • Status changed from new to assigned

comment:4 Changed 2 years ago by sgallagh

  • Status changed from assigned to closed
  • Resolution set to fixed

comment:5 Changed 2 years ago by sgallagh

  • Component changed from SSSD to Async Resolver

comment:6 Changed 2 years ago by jhrozek

  • Red Hat Bugzilla changed from [https://bugzilla.redhat.com/show_bug.cgi?id=805918 805918] to [https://bugzilla.redhat.com/show_bug.cgi?id=805918 805918], [https://bugzilla.redhat.com/show_bug.cgi?id=812387 812387]
Note: See TracTickets for help on using tickets.