aboutsummaryrefslogtreecommitdiffstats
path: root/src/client.c (unfollow)
Commit message (Collapse)AuthorFilesLines
2009-01-30client: check for G_IO_ERR and G_IO_HUPMax Kellermann1-8/+19
When we do not explicitly catch G_IO_ERR and G_IO_HUP, GLib can go into an infinite loop, because it won't deliver the socket error to MPD.
2009-01-25client: use the GIOChannel for I/OMax Kellermann1-40/+110
GIOChannel is more portable than raw read()/write() calls. We're using GIOChannel anyway, because we need it for plugging the client into the GLib main loop. Configure the GIOChannel to the bare minimum: no character set, no buffering.
2009-01-25client: use GLib's win32 IO channel on WIN32Max Kellermann1-0/+5
Use g_io_channel_win32_new_socket() instead of g_io_channel_unix_new() on WIN32.
2009-01-25use g_free() instead of free()Max Kellermann1-3/+3
On some platforms, g_free() must be used for memory allocated by GLib. This patch intends to correct a lot of occurrences, but is probably not complete.
2009-01-21use config_get_positive() instead of manual parsingMax Kellermann1-54/+17
Simplify some code by using config_get_positive(), instead of doing manual parsing and validation each time.
2009-01-17conf: no CamelCase, part IMax Kellermann1-5/+5
Renamed functions, types, variables.
2009-01-10client: remove "expire" event on exitMax Kellermann1-0/+3
2009-01-08added missing explicit config.h includesMax Kellermann1-2/+1
2009-01-07client: use GList instead of dlist.hMax Kellermann1-37/+47
Get rid of the non-portable Linux list library, part I.
2009-01-03client: use GLib instead of utils.hMax Kellermann1-7/+8
2009-01-03client: convert GREETING to a static stringMax Kellermann1-2/+2
The length of GREETING is known at compile time, don't use strlen().
2009-01-01main_notify: renamed source to event_pipe.cMax Kellermann1-1/+1
We are going to migrate away from the concept of notifying the main thread. There should be events sent to it instead. This patch starts a series to implement that.
2008-12-31client: run client_manager_expire() in an idle eventMax Kellermann1-1/+22
With the GLib main loop, the client manager can install its own event in case a client is expired. No need for main.c to call client_manager_expire() manually.
2008-12-31client: remember GLib source idMax Kellermann1-9/+14
Remove the event source from the GMainLoop object in client_set_expired().
2008-12-30main: export the main_loop variableMax Kellermann1-1/+2
Other libraries may need to access the main_loop reference, to add or remove events, or to call g_main_loop_quit().
2008-12-30main: use the GLib main loopMax Kellermann1-68/+71
This is a rather huge patch, which unfortunately cannot be splitted. Instead of using our custom ioops.h library, convert everything to use the GLib main loop.
2008-12-29client: include winsock headers on WIN32Max Kellermann1-2/+8
On Windows, socket declarations reside in winsock.h and ws2tcpip.h. The POSIX headers sys/socket.h etc. are not available.
2008-12-29client: use GLib loggingMax Kellermann1-48/+53
2008-12-24client: always attempt to flush deferred buffersMax Kellermann1-2/+11
When a response is very long (e.g. a large playlist > 100k songs), most of it will end up in the deferred buffers. Filling the deferred queue is very expensive currently, because a new buffer is allocated for every client_write() operation. This may lead to long delays, and the client might give up and disconnect meanwhile. This patch makes MPD attempt to flush the deferred queue as often as possible, to work around this problem. Due to the MPD 0.14 code freeze, we should not optimize the buffering code now.
2008-12-02replaced mpd_printf etc by G_GNUC_PRINTFThomas Jansen1-1/+1
We want to remove gcc.h eventually. This takes care of all the G_GNUC_PRINTF macros.
2008-11-24client.c: replaced mpd_unused by G_GNUC_UNUSEDThomas Jansen1-1/+1
2008-11-22command: allow clients to subscribe to specific idle eventsMarc Pavot1-17/+12
The client may provide the names of idle events as arguments to the "idle" command to inform MPD that it is only interested in these events.
2008-10-31client: use boolMax Kellermann1-1/+1
Return bool instead of int.
2008-10-31client: don't try again after partial writeMax Kellermann1-0/+1
After a partial write, chances are vanishing that another write() will succeed. Don't try immediately.
2008-10-31client: use GQueue instead of sllist.h for deferred_sendMax Kellermann1-34/+42
Another custom data structore converted to GLib.
2008-10-31client: use GSList instead of struct strnode for command listsMax Kellermann1-41/+15
Replace a custom data structure with a GLib one.
2008-10-31client: removed list_cacheMax Kellermann1-34/+1
The list cache aims to save memory allocations, and complicates the code a bit. We should rather use GLib slices later, which are easy to use.
2008-10-22command: no CamelCaseMax Kellermann1-4/+4
Eliminate CamelCase in all public and static functions.
2008-10-17client: converted permissions to unsignedMax Kellermann1-3/+3
client->permission is a bit set, and should be unsigned.
2008-10-17client: eliminate variable "left" in client_write()Max Kellermann1-5/+5
Reduce two temporary variables to only one.
2008-10-17client: fixed send bufferMax Kellermann1-49/+4
There is no sense in using the kernel's send buffer size (SO_SNDBUF) for MPD's send buffer. Convert it into a static buffer of 4 kB.
2008-10-17client: removed CLIENT_MAX_BUFFER_LENGTHMax Kellermann1-3/+2
Use a literal in the struct declaration, and sizeof(client->buffer) everywhere else. Also shrink the buffer from 40 kB to 4 kB. The buffer must only be large enough to hold one line of input, and 4 kB is still more than enough.
2008-10-17client: added assertions on the buffer pointersMax Kellermann1-0/+6
The buffer pointers must not exceed the buffer size.
2008-10-17client: read() return value is ssize_tMax Kellermann1-2/+2
Use ssize_t instead of int.
2008-10-17client: handle partial lines correctlyMax Kellermann1-2/+2
Commit 6eb62e47 didn't obey partial lines correctly: when a line wasn't finished in one read, the first part was ignored when the rest arrived.
2008-10-15listen, client: enable SO_PASSCRED, get client's uidMax Kellermann1-1/+12
Enable authentication over unix sockets. Store the client's uid in the client struct.
2008-10-15idle: client command "noidle" aborts idleMax Kellermann1-3/+21
2008-10-15client: optimized client_input_received()Max Kellermann1-38/+44
Use memchr() instead of manually traversing the input buffer. Update the client's properties after all commands have been processed. Check for buffer overflow once.
2008-10-14command: added command "idle"Max Kellermann1-1/+82
"idle" waits until something noteworthy happens on the server, e.g. song change, playlist modified, database updated. This allows clients to keep up to date without polling.
2008-10-08don't include os_compat.hMax Kellermann1-1/+7
When there are standardized headers, use these instead of the bloated os_compat.h.
2008-09-17client: check expired after client_process_line()Max Kellermann1-1/+3
The assertion on "!client_is_expired(client)" was wrong, because writing the command response may cause the client to become expired. Replace that assertion with a check.
2008-09-10client: simplified client_read()Max Kellermann1-3/+5
Remove one comparison by changing branch order.
2008-09-10client: client_input_received() returns 0Max Kellermann1-4/+2
Since the caller chain doesn't care about the return value (except for COMMAND_RETURN_KILL, COMMAND_RETURN_CLOSE), just return 0 if there is nothing special. This saves one local variable initialization, and one access to it. Also remove one unreachable "return 1" from client_read().
2008-09-10client: check for COMMAND_RETURN_CLOSEMax Kellermann1-15/+14
Don't close the client within client_process_line(), return COMMAND_RETURN_CLOSE instead. This is the signal for the caller chain to actually close it. This makes dealing with the client pointer a lot safer, since the caller always knows whether it is still valid.
2008-09-10client: renamed local variable "selret" to "ret"Max Kellermann1-4/+4
It's easier to reuse the variable if it has a more generic name.
2008-09-10client: moved CLOSE/KILL check after client_process_line()Max Kellermann1-4/+3
Don't update client data if it is going to be closed anyway.
2008-09-07command: use client_[gs]et_permission()Max Kellermann1-3/+1
Don't pass a pointer to client->permission to processCommand(), better let the code in command.c use the new permission getter/setter functions.
2008-09-07client: added client_[gs]et_permission()Max Kellermann1-0/+10
The code in command.c shouldn't mess with a pointer to client->permission. Provide an API for accessing this value.
2008-09-07removed fdprintf() and client_print()Max Kellermann1-25/+0
All callers of fdprintf() have been converted to client_printf() or fprintf(); it is time to remove this clumsy hack now. We can also remove client_print() which took a file descriptor as parameter.
2008-09-07client: removed client_get_fd()Max Kellermann1-8/+0
Now that we have removed all invocations of client_get_fd(), we can safely remove this transitional function. All access to the file descriptor is now hidden behind the interface declared in client.h.