bugGNU Wget - Bugs: bug #46479, null pointer dereference:...

 
 

bug #46479: null pointer dereference: gnutls_free (ctx->session_data->data)

Submitter:  Sami Farin <Safari>
Submitted:  Thu 19 Nov 2015 07:59:57 PM UTC
   
 
Category:  Crash/Freeze/Infloop Severity:  3 - Normal
Priority:  5 - Normal Status:  None
Privacy:  Public Assigned to:  None
Originator Name:  Open/Closed:  Closed
Release:  None Operating System:  None
Reproducibility:  None Fixed Release:  None
Planned Release:  None Regression:  None
Work Required:  None Patch Included:  None
* Mandatory Fields

Add a New Comment Rich Markup
   

Fri 20 Nov 2015 07:06:18 PM UTC, comment #3: 

You "forgot" credits and reference to bug #46479 in the git changelog.

Sami Farin <Safari>
Fri 20 Nov 2015 06:53:05 PM UTC, comment #2: 

thanks, the patch looks correct and I pushed it.

Giuseppe Scrivano <gscrivano>
Group administrator
Fri 20 Nov 2015 05:40:28 PM UTC, comment #1: 

The attached patch should fix it. Thanks for the report.

(file #35506)

Ander Juaristi <ajuaristi>
Thu 19 Nov 2015 07:59:57 PM UTC, original submission:  

I did not read enough code to be able to say conclusively can the deref happen.

if (!ctx || !ctx->session_data || gnutls_session_set_data (session, ctx->session_data->data, ctx->session_data->size))
{
    /* server does not want to continue the session */
    gnutls_free (ctx->session_data->data);

Sami Farin <Safari>

 

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

Attach Files:
   
   
Comment:
   

Attached Files

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by gscrivano (Posted a comment)
  • -email is unavailable- added by ajuaristi (Updated the item)
  • -email is unavailable- added by Safari (Submitted the item)
  •  

    Follow 2 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2015-11-20 gscrivano Open/ClosedOpen Closed
    2015-11-20 ajuaristi Attached File- Added 0001-Fix-potential-NULL-pointer-dereference.patch, #35506

    Back to the top

    Powered by Savane 3.13-02a9.
    Corresponding source code