2025年5月9日 14:19，Patrick Steinhardt <ps@xxxxxx> 写道： > So does that mean that _every_ user of the "parse-options" interfaces > now has to explicitly plug this memory leak when facing unknown options? > That sounds rather undesirable, as there are so many users out there. Since the lifetime of `argv` last until the program terminates, a memory leak can only occur if parse_option is called multiple times and at least two of those calls use the `PARSE_OPT_KEEP_UNKNOWN` flag. In the other words, the memory leak only occurs when the statement `ctx->argue[0] = xstrdup` overwrites the result of a previous `xstrdup` call. > Hm. Is there any other usecase for the `strdup_fn` field that you can > think about in the future? Otherwise it feels a bit overengineered from > my perspective. I think a simple approach is to add a marker to the string allocated by `xstrdup` , and before the next potential leaking `ctx->argv[0] = xstrdup`, check whether the string needs to be freed. Like we could allocate one more byte in the end of the string to store the marker.
