mainGNU Generic Security Service Library - Support: sr #111075, nullptr dereference in...

 
 

sr #111075: nullptr dereference in gss_display_status()

Submitter:  None
Submitted:  Thu 13 Jun 2024 09:21:37 PM UTC
   
 
Category:  None Priority:  5 - Normal
Severity:  5 - Blocker Status:  None
Privacy:  Public Assigned to:  None
Originator Email:  -email is unavailable- Open/Closed:  Open
Operating System:  None
* Mandatory Fields

Post a Comment

Add a New Comment Rich Markup
   

Discussion

Thu 13 Jun 2024 09:21:37 PM UTC, original submission:  

I noticed that two of curl's unit tests (#2077 and #2078, curl v8.8.0) crashed, which I could trace to them calling `gss_display_status()` with `GSS_C_NO_OID` in `curl_gssapi.c` leading to a nullptr dereference on line 336 of gss's `error.c` (v 1.0.4).

Replacing that line with a simple
```
return mech ? mech->display_status(...) : GSS_S_BAD_MECH;
```
prevents the crash.

I can't tell if curl is using the GSS API inappropriately here but I think libgss shouldn't call the result of a `mech` lookup that can fail blindly either.

Anonymous

 

Attached Files

This item currently has no attached files.

(Note: upload size limit is set to 4.0MiB, after insertion of the required escape characters.)

Attach Files:
   
   
Comment:
   

 

Dependencies

This item does not depend on any other items.

No items depend on this one.

 

Mail Notification Carbon-Copy List

Carbon-Copy List
  • -email is unavailable- added by None (Submitted the item)
  •  

    Votes

    There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

    Only logged-in users can vote.

     

    Please enter the title of George Orwell's famous dystopian book (it's a date):

    History

    No changes have been made to this item

    Back to the top

    Powered by Savane 3.16-a7ba.
    Corresponding source code