Project

General

Profile

Actions

Bug #13581

closed

geniconvtbl gets really confused about freeing cpp options

Added by Rich Lowe 3 months ago. Updated 2 months ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
cmd - userland programs
Start date:
Due date:
% Done:

100%

Estimated time:
Difficulty:
Bite-size
Tags:
Gerrit CR:

Description

geniconvtbl keeps a list of options for the C pre-processor, but is very confused about how to free them, and ends up freeing junk.

In short, we end up freeing pointer we allocated, and not the memory we allocated, while also freeing off the end of the list.

Actions #1

Updated by Electric Monk 3 months ago

  • Gerrit CR set to 1285
Actions #2

Updated by Rich Lowe 3 months ago

tested by building under libumem with this geniconvtbl installed, and no longer aborting
(later tested with #13588)

Actions #3

Updated by Electric Monk 2 months ago

  • Status changed from In Progress to Closed
  • % Done changed from 0 to 100

git commit c45618a5778581b6ec976bf7c034fd7afaa757d7

commit  c45618a5778581b6ec976bf7c034fd7afaa757d7
Author: Richard Lowe <richlowe@richlowe.net>
Date:   2021-03-10T19:27:22.000Z

    13581 geniconvtbl gets really confused about freeing cpp options
    Reviewed by: Robert Mustacchi <rm@fingolfin.org>
    Reviewed by: Toomas Soome <tsoome@me.com>
    Approved by: Gordon Ross <gordon.w.ross@gmail.com>

Actions

Also available in: Atom PDF