From mboxrd@z Thu Jan 1 00:00:00 1970 From: Robin Roevens To: development@lists.ipfire.org Subject: Re: [PATCH v2 1/5] misc-progs: addonctrl: Add support for 'Services' metadata Date: Sun, 09 Oct 2022 01:09:42 +0200 Message-ID: <72b37865fd08385ac142b951d8118766df1c1803.camel@sicho.home> In-Reply-To: MIME-Version: 1.0 Content-Type: multipart/mixed; boundary="===============4796992583721875349==" List-Id: --===============4796992583721875349== Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable Hi Michael Thanks again for the review and comments. Michael Tremer schreef op vr 07-10-2022 om 16:01 [+0100]: > Hello again, >=20 > > On 6 Oct 2022, at 18:59, Robin Roevens > > wrote: > >=20 > > * Addonctrl will now check in addon metadata for the exact > > initscript > > =C2=A0names (Services). If more than one initscript is defined for an > > addon, > > =C2=A0the requested action will be performed on all listed initscripts. > > * Added posibility to perform action on a specific initscript of an > > =C2=A0addon instead of on all initscripts of the addon. > > * New action 'list-services' to display a list of services related > > to > > =C2=A0an addon. > > * New action 'boot-status' to display wether service(s) are enabled > > =C2=A0to start on boot or not. > > * More error checking and cleaner error reporting to user > > * General cleanup and code restructuring to avoid code duplication > > * Updated and made usage instructions more verbose. > >=20 > > Fixes: Bug#12935 > > Signed-off-by: Robin Roevens > > --- > > src/misc-progs/addonctrl.c | 397 +++++++++++++++++++++++++++++++--- > > --- > > 1 file changed, 336 insertions(+), 61 deletions(-) > >=20 > > diff --git a/src/misc-progs/addonctrl.c b/src/misc- > > progs/addonctrl.c > > index 14b4b1325..1687aac19 100644 > > --- a/src/misc-progs/addonctrl.c > > +++ b/src/misc-progs/addonctrl.c > > @@ -10,71 +10,346 @@ > > #include > > #include > > #include > > +#include > > #include > > +#include > > +#include > > +#include > > #include "setuid.h" > >=20 > > #define BUFFER_SIZE 1024 > >=20 > > +const char *enabled_path =3D "/etc/rc.d/rc3.d"; > > +const char *disabled_path =3D "/etc/rc.d/rc3.d/off"; > > + > > +char errormsg[BUFFER_SIZE] =3D ""; > > +const char *usage =3D=20 > > +=C2=A0=C2=A0=C2=A0 "Usage\n" > > +=C2=A0=C2=A0=C2=A0 "=C2=A0 addonctrl > > (start|stop|restart|reload|enable|disable|status|boot-status|list- > > services) []\n" > > +=C2=A0=C2=A0=C2=A0 "\n" > > +=C2=A0=C2=A0=C2=A0 "Options:\n" > > +=C2=A0=C2=A0=C2=A0 "=C2=A0 \t\tName of the addon to control\n" > > +=C2=A0=C2=A0=C2=A0 "=C2=A0 \t\tSpecific service of the addon to= control > > (optional)\n" > > +=C2=A0=C2=A0=C2=A0 "=C2=A0 \t\t\tBy default the requested action is perf= ormed on all > > related services. See also 'list-services'.\n" > > +=C2=A0=C2=A0=C2=A0 "=C2=A0 start\t\t\tStart service(s) of the addon\n" > > +=C2=A0=C2=A0=C2=A0 "=C2=A0 stop\t\t\tStop service(s) of the addon\n" > > +=C2=A0=C2=A0=C2=A0 "=C2=A0 restart\t\tRestart service(s) of the addon\n" > > +=C2=A0=C2=A0=C2=A0 "=C2=A0 enable\t\tEnable service(s) of the addon to s= tart at > > boot\n" > > +=C2=A0=C2=A0=C2=A0 "=C2=A0 disable\t\tDisable service(s) of the addon to= start at > > boot\n" > > +=C2=A0=C2=A0=C2=A0 "=C2=A0 status\t\tDisplay current state of addon serv= ice(s)\n" > > +=C2=A0=C2=A0=C2=A0 "=C2=A0 boot-status\t\tDisplay wether service(s) is e= nabled on boot > > or not\n" > > +=C2=A0=C2=A0=C2=A0 "=C2=A0 list-services\t\tDisplay a list of services r= elated to the > > addon"; > > + > > +// Find a file using as glob pattern.=20 > > +// Returns the found filename or NULL if not found > > +char *find_file_in_dir(const char *path, const char *filepattern)=20 > > +{ > > +=C2=A0=C2=A0=C2=A0 struct dirent *entry; > > +=C2=A0=C2=A0=C2=A0 DIR *dp; > > +=C2=A0=C2=A0=C2=A0 char *found =3D NULL; > > + > > +=C2=A0=C2=A0=C2=A0 if ((dp =3D opendir(path)) !=3D NULL) { > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 while(found =3D=3D NULL && (e= ntry =3D readdir(dp)) !=3D NULL) > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 if (f= nmatch(filepattern, entry->d_name, FNM_PATHNAME) > > =3D=3D 0) > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0= =C2=A0=C2=A0=C2=A0 found =3D strdup(entry->d_name); > > + > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 closedir(dp); > > +=C2=A0=C2=A0=C2=A0 } > > + > > +=C2=A0=C2=A0=C2=A0 return found; > > +} > > + > > +// Reads Services metadata for . > > +// Returns pointer to array of strings containing the services for > > =20 > > +// and sets to the number of found services > > +char **get_addon_services(const char *addon, int *servicescnt, > > const char *filter) { > > +=C2=A0=C2=A0=C2=A0 const char *metafile_prefix =3D "/opt/pakfire/db/inst= alled/meta- > > "; > > +=C2=A0=C2=A0=C2=A0 const char *metadata_key =3D "Services"; > > +=C2=A0=C2=A0=C2=A0 const char *keyvalue_delim =3D ":"; > > +=C2=A0=C2=A0=C2=A0 const char *service_delim =3D " "; > > +=C2=A0=C2=A0=C2=A0 char *token; > > +=C2=A0=C2=A0=C2=A0 char **services =3D NULL; > > +=C2=A0=C2=A0=C2=A0 char *service; > > +=C2=A0=C2=A0=C2=A0 char *line =3D NULL; > > +=C2=A0=C2=A0=C2=A0 size_t line_len =3D 0; > > +=C2=A0=C2=A0=C2=A0 int i =3D 0; > > +=C2=A0=C2=A0=C2=A0 char *metafile; > > +=20 > > +=C2=A0=C2=A0=C2=A0 if (addon =3D=3D NULL) { > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 errno =3D EINVAL; > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 return NULL; > > +=C2=A0=C2=A0=C2=A0 } > > + > > +=C2=A0=C2=A0=C2=A0 if (asprintf(&metafile, "%s%s", metafile_prefix, addo= n) =3D=3D -1) > > { > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 errno =3D ENOMEM; > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 return NULL; > > +=C2=A0=C2=A0=C2=A0 } >=20 > You should initialise metafile with NULL before passing it to > asprintf(). I'm not sure why? I can't find any information on why it should be initialized to NULL as it is passed to malloc, which doesn't seem to require it to be NULL beforehand. Or at least I didn't find any documentation stating as such. What I did find is that many implementations of C leave the string in undefined state when it was unable to malloc. While others (like GNU) set it to NULL explicitly if malloc fails. But since I check the return value of asprintf, I never read or touch the returned string, undefined or not, so I didn't think it was required. But I'll initialize them. It's probably indeed the safest to do anyway. >=20 > asprintf() will automatically set errno, actually. Ok. I was not sure as the manpage for asprintf does not mention that while the manpage for for example strdup does mention that explicitly. And I have investigated the source of asprintf, and it does not set errno when there is a problem with malloc..=20 But now I found that malloc itself actually sets errno. So it will indeed be set by asprintf too as that calls malloc. So I'll remove all my setting of ENOMEM. >=20 > > + > > +=C2=A0=C2=A0=C2=A0 FILE *fp =3D fopen(metafile,"r"); > > +=C2=A0=C2=A0=C2=A0 if (fp !=3D NULL) { >=20 > Just FYI, I like writing this check as =E2=80=9Cif (!fp)=E2=80=9D because i= t is a a > lot shorter... It seems there are 2 camps about this, one just writes what you like to write, while the other argues it is more clear that you are working with pointers by explicitly checking against NULL. But if you think that is unnecessary I'll change it. Should I change all my other checks against NULL also? Or only the filepointer ?=20 >=20 > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 // Get initscript(s) for addo= n from meta-file > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 while (getline(&line, &line_l= en, fp) !=3D -1 && services =3D=3D > > NULL) { > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 // St= rip newline > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 char = *newline =3D strchr(line, '\n'); > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 if (n= ewline) *newline =3D 0; >=20 > Yes, this will cut off the trailing newline. >=20 > > + > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 // Pa= rse key/value and look for required key. > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 token= =3D strtok(line, keyvalue_delim); > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 if (t= oken !=3D NULL && strcmp(token, metadata_key) =3D=3D 0) > > { > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0= =C2=A0=C2=A0=C2=A0 token =3D strtok(NULL, keyvalue_delim); > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0= =C2=A0=C2=A0=C2=A0 if (token !=3D NULL) { > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0= =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 // Put each service in services ar= ray > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0= =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 service =3D strtok(token, service_= delim); > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0= =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 while (service !=3D NULL) { > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0= =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 // if filt= er is set, only select filtered > > service > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0= =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 if ((filte= r !=3D NULL && strcmp(filter, > > service) =3D=3D 0) ||=20 > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0= =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2= =A0=C2=A0=C2=A0 filter =3D=3D NULL) { > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0= =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2= =A0=C2=A0=C2=A0 services =3D reallocarray(services ,i+1 > > ,sizeof (char *)); > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0= =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2= =A0=C2=A0=C2=A0 if (services !=3D NULL) > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0= =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2= =A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 services[i++] =3D strdup(service); >=20 > Technically you could check whether strdup() was successful, but I > would accept this version. Indeed, I forgot. As you gave enough comments to need a new version of the patchset, I'll include this too anyway. :-) >=20 > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0= =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2= =A0=C2=A0=C2=A0 else=20 > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0= =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2= =A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 break; > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0= =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 } > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0= =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 service = =3D strtok(NULL, service_delim); > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0= =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 } > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0= =C2=A0=C2=A0=C2=A0 } > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 } > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 } > > + > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 free(line); >=20 > Are you sure that line will always be non-NULL? >=20 > What happens if you read an empty file and getline() exists > immediately without ever allocating a buffer? You are right. Not sure what will happen then. So I'll check if 'line' is not NULL before trying to free it. >=20 > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 fclose(fp); > > +=C2=A0=C2=A0=C2=A0 }=C2=A0 else { > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 snprintf(errormsg, BUFFER_SIZ= E - 1, "Addon '%s' not > > found.\n\n%s", addon, usage); > > +=C2=A0=C2=A0=C2=A0 } > > + > > +=C2=A0=C2=A0=C2=A0 free(metafile); > > +=C2=A0=C2=A0=C2=A0 *servicescnt =3D i; > > +=C2=A0=C2=A0=C2=A0 return services; > > +} > > + > > +// Calls initscript with parameter > > +int initscript_action(const char *service, const char *action) { > > +=C2=A0=C2=A0=C2=A0 const char *initd_path =3D "/etc/rc.d/init.d"; > > +=C2=A0=C2=A0=C2=A0 char *initscript; >=20 > Same here. Please initialise it. >=20 > > +=C2=A0=C2=A0=C2=A0 char *argv[] =3D { > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 action, > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 NULL > > +=C2=A0=C2=A0=C2=A0 }; > > +=C2=A0=C2=A0=C2=A0 int r =3D 0; > > + > > +=C2=A0=C2=A0=C2=A0 if ((r =3D asprintf(&initscript, "%s/%s", initd_path,= service)) > > !=3D -1) { > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 r =3D run(initscript, argv); > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 free(initscript); > > +=C2=A0=C2=A0=C2=A0 } else { > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 errno =3D ENOMEM; > > +=C2=A0=C2=A0=C2=A0 } >=20 > This could look slightly cleaner as: >=20 > =E2=80=A6 >=20 > r =3D asprintf(&initscript, =E2=80=A6.); > if (r > 0) { > =C2=A0 r =3D run(initscript, argv); > } >=20 > if (initscript) > =C2=A0 free(initscript); >=20 > return r; >=20 > Maybe=E2=80=A6 I don=E2=80=99t know. Yes, indeed, as I don't have to set ENOMEM >=20 > > + > > +=C2=A0=C2=A0=C2=A0 return r;=20 > > +} > > + > > +// Move an initscript with filepattern from to > > > > +// Returns: > > +//=C2=A0=C2=A0 -1: Error during move or memory allocation. Details in er= rno > > +//=C2=A0=C2=A0 0: Success > > +//=C2=A0=C2=A0 1: file was not moved, but is already in > > +//=C2=A0=C2=A0 2: file does not exist in either in or > > +int move_initscript_by_pattern(const char *src_path, const char > > *dest_path, const char *filepattern) { > > +=C2=A0=C2=A0=C2=A0 char *src =3D NULL; > > +=C2=A0=C2=A0=C2=A0 char *dest =3D NULL; > > +=C2=A0=C2=A0=C2=A0 int r =3D 1; > > +=C2=A0=C2=A0=C2=A0 char *filename =3D NULL; > > + > > +=C2=A0=C2=A0=C2=A0 if ((filename =3D find_file_in_dir(src_path, filepatt= ern)) !=3D > > NULL) { > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 if ((r =3D asprintf(&src, "%s= /%s", src_path, filename)) !=3D - > > 1 && > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 (r = =3D asprintf(&dest, "%s/%s", dest_path, filename) !=3D > > -1)) { > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 // mo= ve initscript > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 r =3D= rename(src, dest); > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 } else { > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 errno= =3D ENOMEM; > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 } > > + > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 if (src !=3D NULL) > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 free(= src); > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 if (dest !=3D NULL) > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 free(= dest); > > +=C2=A0=C2=A0=C2=A0 } else { > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 if ((filename =3D find_file_i= n_dir(dest_path, filepattern)) > > =3D=3D NULL)=20 > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 r =3D= 2; > > +=C2=A0=C2=A0=C2=A0 } > > + > > +=C2=A0=C2=A0=C2=A0 if (filename !=3D NULL) > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 free(filename); > > + > > +=C2=A0=C2=A0=C2=A0 return r; > > +} > > + > > +// Enable/Disable addon service(s) by moving initscript symlink > > from/to disabled_path > > +int toggle_service(const char *service, const char *action) { > > +=C2=A0=C2=A0=C2=A0 const char *src_path, *dest_path; > > +=C2=A0=C2=A0=C2=A0 char *filepattern;=20 >=20 > Init to NULL. >=20 > > +=C2=A0=C2=A0=C2=A0 int r =3D 0; > > +=C2=A0=C2=A0=C2=A0=20 > > +=C2=A0=C2=A0=C2=A0 if (asprintf(&filepattern, "S??%s", service) =3D=3D -= 1) { > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 errno =3D ENOMEM; > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 return -1; > > +=C2=A0=C2=A0=C2=A0 } > > + > > +=C2=A0=C2=A0=C2=A0 if (strcmp(action, "enable") =3D=3D 0) { > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 src_path =3D disabled_path; > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 dest_path =3D enabled_path; > > +=C2=A0=C2=A0=C2=A0 } else { > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 src_path =3D enabled_path; > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 dest_path =3D disabled_path; > > +=C2=A0=C2=A0=C2=A0 } > > + > > +=C2=A0=C2=A0=C2=A0 // Ensure disabled_path exists > > +=C2=A0=C2=A0=C2=A0 errno =3D 0; > > +=C2=A0=C2=A0=C2=A0 if (mkdir(disabled_path, S_IRWXU + S_IRGRP + S_IXGRP = + S_IROTH > > + S_IXOTH) =3D=3D -1 && errno !=3D EEXIST) { > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 r =3D 1; > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 snprintf(errormsg, BUFFER_SIZ= E -1, "Error creating %s. > > (Error: %d)", disabled_path, errno); > > +=C2=A0=C2=A0=C2=A0 } else { > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 r =3D move_initscript_by_patt= ern(src_path, dest_path, > > filepattern); > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 if (r =3D=3D -1 ) { > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 r =3D= 1; > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 snpri= ntf(errormsg, BUFFER_SIZE - 1, "Could not %s %s. > > (Error: %d)", action, service, errno); > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 } else if (r =3D=3D 1) { > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 snpri= ntf(errormsg, BUFFER_SIZE - 1, "Service %s is > > already %sd. Skipping...", service, action); > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 } else if (r =3D=3D 2) { > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 snpri= ntf(errormsg, BUFFER_SIZE - 1, "Unable to %s > > service %s. (Service has no valid symlink in %s).", action, > > service, src_path); > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 } > > +=C2=A0=C2=A0=C2=A0 } > > + > > +=C2=A0=C2=A0=C2=A0 free(filepattern); > > +=C2=A0=C2=A0=C2=A0=20 > > +=C2=A0=C2=A0=C2=A0 return r; > > +} > > + > > +// Print to stdout wether is enabled or disabled on boot > > +// Prints as Not available when initscript is not found > > +// in either enabled_path or disabled_path. > > +void print_boot_status(char *service) { > > +=C2=A0=C2=A0=C2=A0 char *filepattern; >=20 > NULL. How does this even run without it? >=20 > > +=C2=A0=C2=A0=C2=A0 if (asprintf(&filepattern, "S??%s", service) =3D=3D -= 1) { > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 errno =3D ENOMEM; > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 return; > > +=C2=A0=C2=A0=C2=A0 } > > + > > +=C2=A0=C2=A0=C2=A0 if (find_file_in_dir(enabled_path, filepattern) !=3D = NULL)=20 > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 fprintf(stdout, "%s is enable= d on boot.\n", service); > > +=C2=A0=C2=A0=C2=A0 else if (find_file_in_dir(disabled_path, filepattern)= !=3D NULL) > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 fprintf(stdout, "%s is disabl= ed on boot.\n", service); > > +=C2=A0=C2=A0=C2=A0 else > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 fprintf(stdout, "%s is not av= ailable for boot. (Service > > has no valid symlink in either %s or %s).\n", service, > > enabled_path, disabled_path); > > + > > +=C2=A0=C2=A0=C2=A0 free(filepattern); > > +} > > + > > int main(int argc, char *argv[]) { > > -=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0char command[BUFFER_SIZE]; > > - > > -=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0if (!(initsetuid())) > > -=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0= =C2=A0=C2=A0=C2=A0exit(1); > > - > > -=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0if (argc < 3) { > > -=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0= =C2=A0=C2=A0=C2=A0fprintf(stderr, "\nMissing arguments.\n\naddonctrl > > addon (start|stop|restart|reload|enable|disable)\n\n"); > > -=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0= =C2=A0=C2=A0=C2=A0exit(1); > > -=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0} > > - > > -=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0const char* name =3D argv[1]; > > - > > -=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0if (strlen(name) > 32) { > > -=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 fprintf(std= err, "\nString to large.\n\naddonctrl addon > > (start|stop|restart|reload|enable|disable)\n\n"); > > -=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 exit(1); > > -=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0} > > - > > -=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0// Check if the input argument= is valid > > -=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0if (!is_valid_argument_alnum(n= ame)) { > > -=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0= =C2=A0=C2=A0=C2=A0fprintf(stderr, "Invalid add-on name: %s\n", name); > > -=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0= =C2=A0=C2=A0=C2=A0exit(2); > > -=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0} > > - > > -=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0sprintf(command, "/opt/pakfire= /db/installed/meta-%s", > > name); > > -=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0FILE *fp =3D fopen(command,"r"= ); > > -=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0if ( fp ) { > > -=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 fclose(fp); > > -=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0} else { > > -=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 fprintf(std= err, "\nAddon '%s' not found.\n\naddonctrl > > addon (start|stop|restart|reload|status|enable|disable)\n\n", > > name); > > -=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 exit(1); > > -=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0} > > - > > -=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0if (strcmp(argv[2], "start") = =3D=3D 0) { > > -=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0= =C2=A0=C2=A0=C2=A0snprintf(command, BUFFER_SIZE - 1, > > "/etc/rc.d/init.d/%s start", name); > > -=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0= =C2=A0=C2=A0=C2=A0safe_system(command); > > -=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0} else if (strcmp(argv[2], "st= op") =3D=3D 0) { > > -=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0= =C2=A0=C2=A0=C2=A0snprintf(command, BUFFER_SIZE - 1, > > "/etc/rc.d/init.d/%s stop", name); > > -=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0= =C2=A0=C2=A0=C2=A0safe_system(command); > > -=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0} else if (strcmp(argv[2], "re= start") =3D=3D 0) { > > -=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0= =C2=A0=C2=A0=C2=A0snprintf(command, BUFFER_SIZE - 1, > > "/etc/rc.d/init.d/%s restart", name); > > -=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0= =C2=A0=C2=A0=C2=A0safe_system(command); > > -=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0} else if (strcmp(argv[2], "re= load") =3D=3D 0) { > > -=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0= =C2=A0=C2=A0=C2=A0snprintf(command, BUFFER_SIZE - 1, > > "/etc/rc.d/init.d/%s reload", name); > > -=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0= =C2=A0=C2=A0=C2=A0safe_system(command); > > -=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0} else if (strcmp(argv[2], "st= atus") =3D=3D 0) { > > -=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0= =C2=A0=C2=A0=C2=A0snprintf(command, BUFFER_SIZE - 1, > > "/etc/rc.d/init.d/%s status", name); > > -=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0= =C2=A0=C2=A0=C2=A0safe_system(command); > > -=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0} else if (strcmp(argv[2], "en= able") =3D=3D 0) { > > -=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0= =C2=A0=C2=A0=C2=A0snprintf(command, BUFFER_SIZE - 1, "mv -f > > /etc/rc.d/rc3.d/off/S??%s /etc/rc.d/rc3.d" , name); > > -=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0= =C2=A0=C2=A0=C2=A0safe_system(command); > > -=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0} else if (strcmp(argv[2], "di= sable") =3D=3D 0) { > > -=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0= =C2=A0=C2=A0=C2=A0snprintf(command, BUFFER_SIZE - 1, "mkdir -p > > /etc/rc.d/rc3.d/off"); > > -=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0= =C2=A0=C2=A0=C2=A0safe_system(command); > > -=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0= =C2=A0=C2=A0=C2=A0snprintf(command, BUFFER_SIZE - 1, "mv -f > > /etc/rc.d/rc3.d/S??%s /etc/rc.d/rc3.d/off" , name); > > -=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0= =C2=A0=C2=A0=C2=A0safe_system(command); > > -=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0} else { > > -=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0= =C2=A0=C2=A0=C2=A0fprintf(stderr, "\nBad argument given.\n\naddonctrl > > addon (start|stop|restart|reload|enable|disable)\n\n"); > > -=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0= =C2=A0=C2=A0=C2=A0exit(1); > > -=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0} > > - > > -=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0return 0; > > +=C2=A0=C2=A0=C2=A0 char **services =3D NULL; > > +=C2=A0=C2=A0=C2=A0 int servicescnt =3D 0; > > +=C2=A0=C2=A0=C2=A0 char *addon =3D argv[1]; > > +=C2=A0=C2=A0=C2=A0 char *action =3D argv[2]; > > +=C2=A0=C2=A0=C2=A0 char *service_filter =3D NULL; > > +=C2=A0=C2=A0=C2=A0 int r =3D 0; > > + > > +=C2=A0=C2=A0=C2=A0 if (!(initsetuid())) > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 exit(1); > > + > > +=C2=A0=C2=A0=C2=A0 if (argc < 3) { > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 fprintf(stderr, "\nMissing ar= guments.\n\n%s\n\n", usage); > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 exit(1); > > +=C2=A0=C2=A0=C2=A0 } > > + > > +=C2=A0=C2=A0=C2=A0 if (argc =3D=3D 4 && strcmp(action, "list-services") = !=3D 0) > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 service_filter =3D argv[3]; > > + > > +=C2=A0=C2=A0=C2=A0 if (strlen(addon) > 32) { > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 fprintf(stderr, "\nString too= large.\n\n%s\n\n", usage); > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 exit(1); > > +=C2=A0=C2=A0=C2=A0 } > > + > > +=C2=A0=C2=A0=C2=A0 // Check if the input argument is valid > > +=C2=A0=C2=A0=C2=A0 if (!is_valid_argument_alnum(addon)) { > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 fprintf(stderr, "Invalid add-= on name: %s.\n", addon); > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 exit(2); > > +=C2=A0=C2=A0=C2=A0 } > > + > > +=C2=A0=C2=A0=C2=A0 // Get initscript name(s) from addon metadata > > +=C2=A0=C2=A0=C2=A0 errno =3D 0; >=20 > There is usually no need to reset this. I noticed before, during testing, that, since I do an mkdir in toggle_service, but ignore the error if it is EEXIST, that later in the code after using other functions that could produce errno, errno keeps the value EEXIST if those functions just succeed. And as get_addon_services could return NULL when there are no services but could also return NULL due to an error with errno set. However when no error occurred and the EEXIST error is still in errno, the code would falsely conclude that services is NULL due to an error. So now for safety, I make sure it is set to 0 before I call get_addon_services. >=20 > > +=C2=A0=C2=A0=C2=A0 services =3D get_addon_services(addon, &servicescnt, > > service_filter); > > +=C2=A0=C2=A0=C2=A0 if (services =3D=3D NULL || *services =3D=3D 0) { > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 if (errno !=3D 0) >=20 > This is not reliable error checking. The function should return a > defined value if an error has occurred (e.g. NULL) and an error is > detected, that error should be handled. >=20 > So in this case: >=20 > =C2=A0 services =3D get_addon_services(=E2=80=A6); > =C2=A0 if (!services) { > =C2=A0=C2=A0=C2=A0 printf(=E2=80=9CERROR: =E2=80=A6\=E2=80=9D); >=20 > =C2=A0 } >=20 > =C2=A0 =E2=80=A6 Other code, on success=E2=80=A6 >=20 > Now, NULL is not a very good indicator because your problem might be > that an error has been found and therefore the function aborted, or > that there is actually no services available for this package. >=20 > You could handle this by setting errno to something like ENOENT (No > such file or directory) and handle that separately: >=20 > =C2=A0 services =3D get_addon_services(=E2=80=A6); > =C2=A0 if (!services) { > =C2=A0=C2=A0=C2=A0 switch (errno) { > =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 case ENOENT: > =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 DO SOMETHING; > =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 Break; >=20 > =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 default: > =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 printf(=E2=80=9CERROR: =E2=80=A6= \=E2=80=9D); > =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 DO SOMETING ELSE > =C2=A0 } I'm not sure why it is not reliable? I had thought about setting errno before; but I saw it being strongly discouraged in many discussions on the web; unless you where writing an actual system library. So I went for my own 'errormsg'; And I check if errno is set for system errors, if not, check my own errormsg for function errors. If both are clear, services is NULL due to no services. (which could then be due to an invalid filter, or just no services for the addon) >=20 > Finally, instead of printing the errno number in the error message, > you can use %m (without any argument) which will be automatically > converted into something human readable. Are you sure? As I found this: > The %m directive is not portable, use %s mapped to an argument of > strerror(errno) (or a version of strerror_r) instead.=20 here: https://www.gnu.org/software/gnulib/manual/html_node/asprintf.html >=20 > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 fprin= tf(stderr, "\nSystem error occured. (Error: > > %d)\n\n", errno); > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 else if (strcmp(errormsg, "")= !=3D 0) > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 fprin= tf(stderr, "\n%s\n\n", errormsg); > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 else if (service_filter !=3D = NULL) > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 fprin= tf(stderr, "\nNo service '%s' found for addon > > '%s'. Use 'list-services' to get a list of available > > services\n\n%s\n\n", service_filter, addon, usage); > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 else > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 fprin= tf(stderr, "\nAddon '%s' has no services.\n\n", > > addon); > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 exit(1); > > +=C2=A0=C2=A0=C2=A0 } > > + > > +=C2=A0=C2=A0=C2=A0 // Handle requested action > > +=C2=A0=C2=A0=C2=A0 if (strcmp(action, "start") =3D=3D 0 || > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 strcmp(action, "stop") =3D=3D= 0 || > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 strcmp(action, "restart") =3D= =3D 0 || > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 strcmp(action, "reload") =3D= =3D 0 || > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 strcmp(action, "status") =3D= =3D 0) { > > + > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 errno =3D 0; > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 for(int i =3D 0; i < services= cnt; i++) { > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 if (i= nitscript_action(services[i], action) !=3D 0) { > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0= =C2=A0=C2=A0=C2=A0 r =3D 1; > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0= =C2=A0=C2=A0=C2=A0 if (errno !=3D 0)=20 > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0= =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 fprintf(stderr, "\nSystem error oc= cured. > > (Error: %d)\n\n", errno); > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0= =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 break; > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 } > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 } > > + > > +=C2=A0=C2=A0=C2=A0 } else if (strcmp(action, "enable") =3D=3D 0 || > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0= =C2=A0=C2=A0 strcmp(action, "disable") =3D=3D 0) { > > + > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 errno =3D 0; > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 for(int i =3D 0; i < services= cnt; i++) { > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 if (t= oggle_service(services[i], action) =3D=3D 0) { > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0= =C2=A0=C2=A0=C2=A0 fprintf(stdout, "%sd service %s\n", action, > > services[i]); >=20 > If you want to just print something, just use printf. Ok >=20 > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 } els= e if (errno !=3D 0) { > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0= =C2=A0=C2=A0=C2=A0 r =3D 1; > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0= =C2=A0=C2=A0=C2=A0 fprintf(stderr, "\nSystem error occured. (Error: > > %d)\n\n", errno); > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0= =C2=A0=C2=A0=C2=A0 break; > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 } els= e { > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0= =C2=A0=C2=A0=C2=A0 r =3D 1; > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0= =C2=A0=C2=A0=C2=A0 fprintf(stderr, "\n%s\n\n", errormsg); > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 } > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 } > > + > > +=C2=A0=C2=A0=C2=A0 } else if (strcmp(action, "boot-status") =3D=3D 0) { > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 errno =3D 0; > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 for(int i =3D 0; i < services= cnt; i++) { > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 print= _boot_status(services[i]); > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 if (e= rrno !=3D 0) { > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0= =C2=A0=C2=A0=C2=A0 r =3D 1; > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0= =C2=A0=C2=A0=C2=A0 fprintf(stderr, "\nSystem error occured. (Error: > > %d)\n\n", errno); > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0= =C2=A0=C2=A0=C2=A0 break; > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 } > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 } > > +=C2=A0=C2=A0=C2=A0=20 > > +=C2=A0=C2=A0=C2=A0 } else if (strcmp(action, "list-services") =3D=3D 0) { > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 fprintf(stdout, "\nServices f= or addon %s:\n", addon); > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 for(int i =3D 0; i < services= cnt; i++) { > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 fprin= tf(stdout, "=C2=A0 %s\n", services[i]); > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 } > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 fprintf(stdout, "\n"); > > + > > +=C2=A0=C2=A0=C2=A0 } else { > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 fprintf(stderr, "\nBad argume= nt given.\n\n%s\n\n", usage); > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 r =3D 1; > > +=C2=A0=C2=A0=C2=A0 } > > + > > +=C2=A0=C2=A0=C2=A0 // Cleanup > > +=C2=A0=C2=A0=C2=A0 for(int i =3D 0; i < servicescnt; i++)=20 > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 free(services[i]); > > +=C2=A0=C2=A0=C2=A0 free(services); > > + > > +=C2=A0=C2=A0=C2=A0 return r; > > } >=20 > Otherwise, this looks a lot better since the last version :) Thanks. I'm learning a lot. Robin >=20 > -Michael >=20 > > --=20 > > 2.37.3 > >=20 > >=20 > > --=20 > > Dit bericht is gescanned op virussen en andere gevaarlijke > > inhoud door MailScanner en lijkt schoon te zijn. > >=20 >=20 >=20 --=20 Dit bericht is gescanned op virussen en andere gevaarlijke inhoud door MailScanner en lijkt schoon te zijn. --===============4796992583721875349==--