This is the mail archive of the gdb-patches@sources.redhat.com mailing list for the GDB project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [RFA] solib-svr4.c fetch link map address


On Oct 1, 10:28pm, Elena Zannoni wrote:

> +              discard_cleanups (old_chain);
> +              return lm;
> +            }
> +        }
> +      /* Not the file we wanted, continue checking.  */
> +      lm = extract_address (objfile_lm_info.lm + lmo->l_next_offset,
> +			    lmo->l_next_size);
> +      discard_cleanups (old_chain);
> +    }

Why are the cleanups being discarded?  Won't this result in a memory
leak?

Another concern is that there appears to be some duplication of code
between svr4_current_sos() and the function that you've just written. 
I'm wondering if some sort of factoring could be done to minimize
duplication.

Finally, I'm curious about how often we'll be fetching the link
map address.  Is it the case that it'll be fetched once (per
objfile) and never fetched again?  Or will it be fetched repeatedly?
If it's the former, I think your approach is fine.  If the latter, we
should consider saving the link map address so that it can be supplied
to glibc without having to read the target.

Kevin


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]