Hi, thanks for the review.
On Fri, Sep 11, 2020 at 09:03:43PM +0200, David Sommerseth wrote:
> On 11/08/2020 12:44, Gert Doering wrote:
> > This is a new "samples" plugin which does not do many useful things,
> > besides
[..]
> > +sample-client-connect.o: sample-client-connect.c
> > +
> > +# This directory is where we will look for openvpn-plugin.h
> > +CPPFLAGS=-I../../../include
> > +
> > +CC=gcc
>
> If you don't set this; CC will normally be 'cc', which is normally linked to
> gcc on Linux systems.
>
> > +CFLAGS=-O2 -Wall -Wno-unused-variable -g
>
> I would probably skip the -Wno-unused-variable and rather not declare unused
> variables.
Yeah, not sure how that one slipped in. I *did* try to make it warning-free,
so, next version is coming :-)
> > + rl->name = strdup("config");
>
> I'm getting a lot of "warning: implicit declaration of function
> ???strdup???;" and
> "warning: assignment to ???char *??? from ???int??? makes pointer from
> integer
> without a cast" compiler warning on all of these strdup() calls.
>
> We do use strdup() in the main openvpn code, but that code includes config.h,
> which contains #define _GNU_SOURCE 1. This removes this compiler warning.
>
> This is on RHEL-7 with both gcc-4.8 and gcc-9.3.
Not sure what you are hitting there. According to "man strdup()",
inclusion of <string.h> should be fully sufficient, without any extra
declarations. But maybe it's a -std=... thing to tell GCC what
C language level is desired?
I developed and tested this on a gentoo linux, with gcc-9.2.0, and it's
*not* throwing strdup() warnings.
> Otherwise, the code looks reasonable and it works. The log file does not
> include the pushed echo statement (can be enabled in options.c:5286). The
> management interface shows the pushed echo message.
Yeah, the "echo" directive is sort of weird. It's blanked out of PUSH
messages as well... I do wonder what James had in mind here, or what
AS/Connect are using it for :-)
gert
--
"If was one thing all people took for granted, was conviction that if you
feed honest figures into a computer, honest figures come out. Never doubted
it myself till I met a computer with a sense of humor."
Robert A. Heinlein, The Moon is a Harsh Mistress
Gert Doering - Munich, Germany [email protected]
signature.asc
Description: PGP signature
_______________________________________________ Openvpn-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/openvpn-devel
