diff options
author | Jeremy Allison <jra@samba.org> | 2007-11-20 17:57:47 -0800 |
---|---|---|
committer | Jeremy Allison <jra@samba.org> | 2007-11-20 17:57:47 -0800 |
commit | a0f7c3f481e22f77a6f868970f83e27a0d6ba4ad (patch) | |
tree | ab9c7f3c64e098920811d8c21f931fcd093289bf /source3/services/svc_rcinit.c | |
parent | 3c561043dcecf038f5e06d84173b320953fa6652 (diff) | |
download | samba-a0f7c3f481e22f77a6f868970f83e27a0d6ba4ad.tar.gz samba-a0f7c3f481e22f77a6f868970f83e27a0d6ba4ad.tar.xz samba-a0f7c3f481e22f77a6f868970f83e27a0d6ba4ad.zip |
Remove pstring from services/*.c
Jeremy.
(This used to be commit 33aa866195e232f878230bc62e97e1484996afcc)
Diffstat (limited to 'source3/services/svc_rcinit.c')
-rw-r--r-- | source3/services/svc_rcinit.c | 57 |
1 files changed, 36 insertions, 21 deletions
diff --git a/source3/services/svc_rcinit.c b/source3/services/svc_rcinit.c index f65015b0594..66f89f2248f 100644 --- a/source3/services/svc_rcinit.c +++ b/source3/services/svc_rcinit.c @@ -24,20 +24,25 @@ static WERROR rcinit_stop( const char *service, SERVICE_STATUS *status ) { - pstring command; + char *command = NULL; int ret, fd; - - pstr_sprintf( command, "%s/%s/%s stop", dyn_LIBDIR, SVCCTL_SCRIPT_DIR, service ); - + + if (asprintf(&command, "%s/%s/%s stop", + dyn_LIBDIR, SVCCTL_SCRIPT_DIR, service) < 0) { + return WERR_NOMEM; + } + /* we've already performed the access check when the service was opened */ - + become_root(); ret = smbrun( command , &fd ); unbecome_root(); - + DEBUGADD(5, ("rcinit_start: [%s] returned [%d]\n", command, ret)); close(fd); - + + SAFE_FREE(command); + ZERO_STRUCTP( status ); status->type = 0x0020; status->state = (ret == 0 ) ? 0x0001 : 0x0004; @@ -51,19 +56,24 @@ static WERROR rcinit_stop( const char *service, SERVICE_STATUS *status ) static WERROR rcinit_start( const char *service ) { - pstring command; + char *command = NULL; int ret, fd; - - pstr_sprintf( command, "%s/%s/%s start", dyn_LIBDIR, SVCCTL_SCRIPT_DIR, service ); - + + if (asprintf(&command, "%s/%s/%s start", + dyn_LIBDIR, SVCCTL_SCRIPT_DIR, service) < 0) { + return WERR_NOMEM; + } + /* we've already performed the access check when the service was opened */ - + become_root(); ret = smbrun( command , &fd ); unbecome_root(); - + DEBUGADD(5, ("rcinit_start: [%s] returned [%d]\n", command, ret)); - close(fd); + close(fd); + + SAFE_FREE(command); return ( ret == 0 ) ? WERR_OK : WERR_ACCESS_DENIED; } @@ -73,22 +83,27 @@ static WERROR rcinit_start( const char *service ) static WERROR rcinit_status( const char *service, SERVICE_STATUS *status ) { - pstring command; + char *command = NULL; int ret, fd; - - pstr_sprintf( command, "%s/%s/%s status", dyn_LIBDIR, SVCCTL_SCRIPT_DIR, service ); - + + if (asprintf(&command, "%s/%s/%s status", + dyn_LIBDIR, SVCCTL_SCRIPT_DIR, service) < 0) { + return WERR_NOMEM; + } + /* we've already performed the access check when the service was opened */ /* assume as return code of 0 means that the service is ok. Anything else is STOPPED */ - + become_root(); ret = smbrun( command , &fd ); unbecome_root(); - + DEBUGADD(5, ("rcinit_start: [%s] returned [%d]\n", command, ret)); close(fd); - + + SAFE_FREE(command); + ZERO_STRUCTP( status ); status->type = 0x0020; status->state = (ret == 0 ) ? 0x0004 : 0x0001; |