Re: [PATCH v4 1/6] config: Fix free in 'config get' implementation.

Subject: Re: [PATCH v4 1/6] config: Fix free in 'config get' implementation.

Date: Tue, 24 Apr 2012 23:29:27 -0300

To: Peter Wang, notmuch@notmuchmail.org

Cc:

Peter Wang <novalazy@gmail.com> writes:

> The array returned by g_key_file_get_string_list() should be freed with
> g_strfreev(), not free().

Pushed this one patch from the series.

d

Previous message (by thread): Re: [PATCH v4 2/6] config: Check 'config get' arity exactly

Thread:

Peter Wang—[PATCH 1/2] config: Add 'config list' command [inbox, notmuch::patch, notmuch::wip, unread]
- Peter Wang—[PATCH 2/2] man: Document 'config list' command [inbox, notmuch::patch, notmuch::wip, unread]
- Tomi Ollila—Re: [PATCH 1/2] config: Add 'config list' command [inbox, unread]
  - Peter Wang—Re: [PATCH 1/2] config: Add 'config list' command [inbox, unread]
  - Jameson Graef Rollins—Re: [PATCH 1/2] config: Add 'config list' command [inbox, signed, unread]
- Jameson Graef Rollins—Re: [PATCH 1/2] config: Add 'config list' command [inbox, signed, unread]
- Peter Wang—[PATCH v2] Add 'config list' command [inbox, unread]
  - Peter Wang—[PATCH v2 1/5] config: Fix free in 'config get' implementation. [inbox, notmuch::obsolete, notmuch::patch, unread]
  - Peter Wang—[PATCH v2 2/5] test: Add tests for 'config' command [inbox, notmuch::obsolete, notmuch::patch, unread]
    - Mark Walters—Re: [PATCH v2 2/5] test: Add tests for 'config' command [inbox, notmuch::review, unread]
      - Jameson Graef Rollins—Re: [PATCH v2 2/5] test: Add tests for 'config' command [inbox, signed, unread]
  - Peter Wang—[PATCH v2 3/5] test: Add broken test for 'config list' [inbox, notmuch::obsolete, notmuch::patch, unread]
  - Peter Wang—[PATCH v2 4/5] config: Add 'config list' command [inbox, notmuch::obsolete, notmuch::patch, unread]
  - Peter Wang—[PATCH v2 5/5] man: Document 'config list' command [inbox, notmuch::obsolete, notmuch::patch, unread]
- Peter Wang—[PATCH v3 0/5] Add 'config list' command [inbox, unread]
  - Peter Wang—[PATCH v3 1/5] config: Fix free in 'config get' implementation. [inbox, notmuch::obsolete, notmuch::patch, unread]
  - Peter Wang—[PATCH v3 2/5] test: Add tests for 'config' command [inbox, notmuch::obsolete, notmuch::patch, unread]
    - Jameson Graef Rollins—Re: [PATCH v3 2/5] test: Add tests for 'config' command [inbox, signed, unread]
  - Peter Wang—[PATCH v3 3/5] test: Add broken test for 'config list' [inbox, notmuch::obsolete, notmuch::patch, unread]
  - Peter Wang—[PATCH v3 4/5] config: Add 'config list' command [inbox, notmuch::obsolete, notmuch::patch, unread]
    - Jameson Graef Rollins—Re: [PATCH v3 4/5] config: Add 'config list' command [inbox, signed, unread]
      - Peter Wang—Re: [PATCH v3 4/5] config: Add 'config list' command [inbox, unread]
  - Peter Wang—[PATCH v3 5/5] man: Document 'config list' command [inbox, notmuch::obsolete, notmuch::patch, unread]
- Peter Wang—[PATCH v4 0/6] Config-related patches [inbox, unread]
  - Peter Wang—[PATCH v4 1/6] config: Fix free in 'config get' implementation. [inbox, notmuch::patch, notmuch::pushed, notmuch::trivial, unread]
    - David Bremner—Re: [PATCH v4 1/6] config: Fix free in 'config get' implementation. [inbox, unread]
  - Peter Wang—[PATCH v4 2/6] config: Check 'config get' arity exactly [inbox, notmuch::patch, notmuch::pushed, unread]
    - David Bremner—Re: [PATCH v4 2/6] config: Check 'config get' arity exactly [inbox, unread]
  - Peter Wang—[PATCH v4 3/6] test: Add tests for 'config' command [inbox, notmuch::patch, notmuch::pushed, unread]
  - Peter Wang—[PATCH v4 4/6] test: Add broken test for 'config list' [inbox, notmuch::patch, notmuch::pushed, unread]
  - Peter Wang—[PATCH v4 5/6] config: Add 'config list' command [inbox, notmuch::patch, notmuch::pushed, unread]
  - Peter Wang—[PATCH v4 6/6] man: Document 'config list' command [inbox, notmuch::patch, notmuch::pushed, unread]
  - Mark Walters—Re: [PATCH v4 0/6] Config-related patches [inbox, notmuch::review, unread]
    - Jameson Graef Rollins—Re: [PATCH v4 0/6] Config-related patches [inbox, signed, unread]
    - Mark Walters—Re: [PATCH v4 0/6] Config-related patches [inbox, unread]
  - Jameson Graef Rollins—Re: [PATCH v4 0/6] Config-related patches [inbox, signed, unread]