diff options
author | Danny Holman <dholman@gymli.org> | 2023-02-15 22:51:35 -0600 |
---|---|---|
committer | Danny Holman <dholman@gymli.org> | 2023-02-15 22:51:35 -0600 |
commit | 43da00aa155de252928a216a43889cbee7138d0e (patch) | |
tree | b787a28ce435951afca0b2302b8ca2a1159926cd /server/src | |
parent | 89c48e763ef042c9faee5b7c1434dbcaf12da399 (diff) |
session: simplify read_session
Simplify the function read_session and provide some error checking. This
function should now return -1 if a valid session with the given ID
cannot be found.
Signed-off-by: Danny Holman <dholman@gymli.org>
Diffstat (limited to 'server/src')
-rw-r--r-- | server/src/session.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/server/src/session.c b/server/src/session.c index 74b29a7..e01dc34 100644 --- a/server/src/session.c +++ b/server/src/session.c @@ -35,6 +35,7 @@ void write_session(int id, const char *data, size_t size) { size_t read_session(int id, char *data, size_t size) { struct session *ses = find_session(id); - size_t bytes = read(ses->socket, data, size); - return bytes; + if (ses == NULL) + return -1; + return read(ses->socket, data, size); } |