Hi,
On Tue, May 18, 2021 at 10:33 AM Gert Doering <[email protected]> wrote:
>
> Inline peer-fingerprint blocks can benefit from a bit of structuring
> by indentation or by putting comments ("# this is Alice's key").
>
> Signed-off-by: Gert Doering <[email protected]>
> ---
> src/openvpn/options.c | 9 +++++++--
> 1 file changed, 7 insertions(+), 2 deletions(-)
>
> diff --git a/src/openvpn/options.c b/src/openvpn/options.c
> index db460796..cfc30193 100644
> --- a/src/openvpn/options.c
> +++ b/src/openvpn/options.c
> @@ -1145,8 +1145,13 @@ parse_hash_fingerprint_multiline(const char *str, int
> nbytes, int msglevel,
> const char *line;
> while ((line = strsep(&lines, "\n")))
> {
> - /* skip empty lines */
> - if (strlen(line) == 0)
> + /* ignore leading whitespace */
> + while(isspace(*line))
> + {
> + line++;
> + }
> + /* skip empty lines and comment lines */
> + if (strlen(line) == 0 || *line == '#')
As we support two comment characters ('#' and ';'), would be better to
do the same here too. One could relax the requirement of comment as
the first character using parse_line(), but this looks generous
enough.
Selva
_______________________________________________
Openvpn-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/openvpn-devel
