[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCHv2 2/2] bridge helper: support conf dirs
From: |
Stefan Hajnoczi |
Subject: |
Re: [Qemu-devel] [PATCHv2 2/2] bridge helper: support conf dirs |
Date: |
Tue, 5 Mar 2013 10:19:23 +0100 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
On Sat, Mar 02, 2013 at 12:58:48AM -0600, Doug Goldstein wrote:
> @@ -152,6 +199,14 @@ static int parse_acl_file(const char *filename, ACLList
> *acl_list)
> failure:
> fclose(f);
>
> + if (include_list) {
> + for (i = 0; i < include_count; i++) {
> + if (include_list[i])
> + free(include_list[i]);
free(NULL) is a nop so the if isn't necessary - you can
free(include_list[i]) unconditionally.
- [Qemu-devel] [PATCHv2 0/2] bridge helper: includedir conf arg, Doug Goldstein, 2013/03/02
- [Qemu-devel] [PATCHv2 2/2] bridge helper: support conf dirs, Doug Goldstein, 2013/03/02
- [Qemu-devel] [PATCHv3 0/2] bridge helper: includedir conf arg, Doug Goldstein, 2013/03/07
- [Qemu-devel] [PATCHv3 1/2] bridge helper: unified error cleanup for parse_acl_file, Doug Goldstein, 2013/03/07
- [Qemu-devel] [PATCHv3 2/2] bridge helper: support conf dirs, Doug Goldstein, 2013/03/07
- Re: [Qemu-devel] [PATCHv3 0/2] bridge helper: includedir conf arg, Stefan Hajnoczi, 2013/03/07
- Re: [Qemu-devel] [PATCHv3 0/2] bridge helper: includedir conf arg, Corey Bryant, 2013/03/07
- [Qemu-devel] [PATCH v3 0/2] bridge helper: includedir conf arg, Doug Goldstein, 2013/03/18