Hi Dave, Could you re-indent your code. You've created a lot of unnecessary diffs in your patch and there is too much interferences for a clean review.
i.e. On 07/07/2015 09:21 PM, Dave Zhu (yanbzhu) wrote: > - if (bind_conf->default_ctx) { > - memprintf(err, "%sthis version of openssl cannot load multiple > SSL > certificates.\n", > - err && *err ? *err : ""); > - return 1; > - } > + if (bind_conf->default_ctx) { > + memprintf(err, "%sthis version of openssl cannot load > multiple SSL certificates.\n", > + err && *err ? *err : ""); > + goto end; > + } R, Emeric