diff options
| author | Dmitry V. Levin <ldv@altlinux.org> | 2020-01-17 23:46:42 +0000 |
|---|---|---|
| committer | Dmitry V. Levin <ldv@altlinux.org> | 2020-01-20 12:28:59 +0000 |
| commit | 98244ec036096c7571d3bc3782baccab4c7565db (patch) | |
| tree | a865e431999514722799024be58cfdf7666534ab /libpam | |
| parent | c006f4c5a25c95fd4f39dcb87271825d0c154203 (diff) | |
_pam_mkargv: add const qualifier to the first argument
Also fix the following compilation warning:
tests/tst-pam_mkargv.c:21:22: warning: initialization discards ‘const’
qualifier from pointer target type [-Wdiscarded-qualifiers]
char *argvstring = "user = XENDT\\userα user=XENDT\\user1";
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
* libpam/pam_misc.c (_pam_mkargv): Add const qualifier to the first
argument.
* libpam/pam_private.h (_pam_mkargv): Likewise.
* tests/tst-pam_mkargv.c (main): Convert argvstring from a pointer into
a static const string, make argvresult array static const.
Diffstat (limited to 'libpam')
| -rw-r--r-- | libpam/pam_misc.c | 2 | ||||
| -rw-r--r-- | libpam/pam_private.h | 2 |
2 files changed, 2 insertions, 2 deletions
diff --git a/libpam/pam_misc.c b/libpam/pam_misc.c index aac0e923..996f23ce 100644 --- a/libpam/pam_misc.c +++ b/libpam/pam_misc.c @@ -163,7 +163,7 @@ char *_pam_memdup(const char *x, int len) /* Generate argv, argc from s */ /* caller must free(argv) */ -int _pam_mkargv(char *s, char ***argv, int *argc) +int _pam_mkargv(const char *s, char ***argv, int *argc) { int l; int argvlen = 0; diff --git a/libpam/pam_private.h b/libpam/pam_private.h index 8cb77528..ed02bb02 100644 --- a/libpam/pam_private.h +++ b/libpam/pam_private.h @@ -271,7 +271,7 @@ char *_pam_strdup(const char *s); char *_pam_memdup(const char *s, int len); -int _pam_mkargv(char *s, char ***argv, int *argc); +int _pam_mkargv(const char *s, char ***argv, int *argc); void _pam_sanitize(pam_handle_t *pamh); |