#3922: Patch to let filters pass on environment variables
---------------------+----------------------
  Reporter:  rsmmr   |      Owner:  kevin8t8
      Type:  defect  |     Status:  closed
  Priority:  minor   |  Milestone:
 Component:  mutt    |    Version:  1.8.0
Resolution:  fixed   |   Keywords:
---------------------+----------------------
Comment (by Kevin McCarthy <kevin@…>):

 In [changeset:"7cefa378ab7e144216fdfef4e993ffa766f39196"
 6974:7cefa378ab7e]:
 {{{
 #!CommitTicketReference repository=""
 revision="7cefa378ab7e144216fdfef4e993ffa766f39196"
 Fix setenv overwriting to not truncate the envlist. (see #3922)

 The refactor in 2b9c40f13e13 exposed a bug I hadn't noticed.  The
 match loop performed a FREE() on the slot.  Then, below, it was
 checking if (*envp) to see whether it was overwriting or creating a
 new slot.  However, FREE() nulls out *envp.  This would end up
 truncating the envlist just after the set slot!

 Move the free down, using a mutt_str_replace(), when overwriting the
 slot.
 }}}

--
Ticket URL: <https://dev.mutt.org/trac/ticket/3922#comment:5>
Mutt <http://www.mutt.org/>
The Mutt mail user agent

Reply via email to