diff options
| author | Max Kellermann <max@duempel.org> | 2008-09-07 19:15:45 +0200 |
|---|---|---|
| committer | Max Kellermann <max@duempel.org> | 2008-09-07 19:15:45 +0200 |
| commit | bf6994d8e3dde638ced26f4e1ae850422631ee72 (patch) | |
| tree | 77e40b4747be3d6569edb704893fd131e9c73042 /src/command.c | |
| parent | 18fc10a92647f06aba065b80a4de219facf5d550 (diff) | |
| download | mpd-bf6994d8e3dde638ced26f4e1ae850422631ee72.tar.gz mpd-bf6994d8e3dde638ced26f4e1ae850422631ee72.tar.xz mpd-bf6994d8e3dde638ced26f4e1ae850422631ee72.zip | |
command: don't pass permission as pointer where appropriate
Some functions don't want to modify a client's permission set. Pass
the permissions to them by value, not by reference.
Diffstat (limited to 'src/command.c')
| -rw-r--r-- | src/command.c | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/src/command.c b/src/command.c index ef151a872..381efad68 100644 --- a/src/command.c +++ b/src/command.c @@ -153,7 +153,7 @@ static const char check_non_negative[] = "\"%s\" is not an integer >= 0"; static const char *current_command; static int command_listNum; -static CommandEntry *getCommandEntryFromString(char *string, int *permission); +static CommandEntry *getCommandEntryFromString(char *string, int permission); static List *commandList; @@ -833,7 +833,7 @@ static int listHandleUpdate(struct client *client, insertInList(pathList, "", NULL); if (next) - nextCmd = getCommandEntryFromString(next->data, permission); + nextCmd = getCommandEntryFromString(next->data, *permission); if (cmd != nextCmd) { int ret = updateInit(pathList); @@ -1418,7 +1418,7 @@ static int checkArgcAndPermission(CommandEntry * cmd, struct client *client, } static CommandEntry *getCommandEntryAndCheckArgcAndPermission(struct client *client, - int *permission, + int permission, int argc, char *argv[]) { @@ -1439,14 +1439,14 @@ static CommandEntry *getCommandEntryAndCheckArgcAndPermission(struct client *cli current_command = cmd->cmd; - if (checkArgcAndPermission(cmd, client, *permission, argc, argv) < 0) { + if (checkArgcAndPermission(cmd, client, permission, argc, argv) < 0) { return NULL; } return cmd; } -static CommandEntry *getCommandEntryFromString(char *string, int *permission) +static CommandEntry *getCommandEntryFromString(char *string, int permission) { CommandEntry *cmd; char *argv[COMMAND_ARGV_MAX] = { NULL }; @@ -1475,7 +1475,7 @@ static int processCommandInternal(struct client *client, if (argc == 0) return 0; - if ((cmd = getCommandEntryAndCheckArgcAndPermission(client, permission, + if ((cmd = getCommandEntryAndCheckArgcAndPermission(client, *permission, argc, argv))) { if (!cmdnode || !cmd->listHandler) { ret = cmd->handler(client, permission, argc, argv); |