From 9aa3990e50efffc8a2f76e6c55ad769e5f711602 Mon Sep 17 00:00:00 2001 From: "Luke T. Shumaker" Date: Thu, 3 Apr 2025 16:16:33 -0600 Subject: lib9p: srv: Whoops, add a missing `return` --- lib9p/srv.c | 1 + 1 file changed, 1 insertion(+) (limited to 'lib9p/srv.c') diff --git a/lib9p/srv.c b/lib9p/srv.c index bfeb06f..6e6a91b 100644 --- a/lib9p/srv.c +++ b/lib9p/srv.c @@ -903,6 +903,7 @@ static void handle_Topen(struct _lib9p_srv_req *ctx, if (!srv_util_check_perm(ctx, &stat, perm_bits)) { lib9p_error(&ctx->ctx.basectx, LINUX_EACCES, "permission denied"); + return; } /* Actually make the call. */ -- cgit v1.2.3-2-g168b From 8a39cfe7f245bea8b3a458238b694de13c61b88a Mon Sep 17 00:00:00 2001 From: "Luke T. Shumaker" Date: Sat, 5 Apr 2025 03:41:44 -0600 Subject: SYSTEM headers don't get strict GCC checks, so change them to PUBLIC --- lib9p/srv.c | 12 ++++++++++++ 1 file changed, 12 insertions(+) (limited to 'lib9p/srv.c') diff --git a/lib9p/srv.c b/lib9p/srv.c index 6e6a91b..12556b0 100644 --- a/lib9p/srv.c +++ b/lib9p/srv.c @@ -199,7 +199,11 @@ static void respond_error(struct _lib9p_srv_req *req) { LIB9P_TYP_Rerror, &host, &net); +#pragma GCC diagnostic push +#pragma GCC diagnostic ignored "-Wformat" +#pragma GCC diagnostic ignored "-Wformat-extra-args" infof("< %v", lo_box_lib9p_msg_Rerror_as_fmt_formatter(&host)); +#pragma GCC diagnostic pop r = write_Rmsg(req, &net); if (r < 0) nonrespond_errorf("write: %s", net_strerror(-r)); @@ -425,7 +429,11 @@ static void handle_message(struct _lib9p_srv_req *ctx) { enum lib9p_msg_type typ; lib9p_Tmsg_unmarshal(&ctx->ctx.basectx, ctx->net_bytes, &typ, host_req); +#pragma GCC diagnostic push +#pragma GCC diagnostic ignored "-Wformat" +#pragma GCC diagnostic ignored "-Wformat-extra-args" infof("> %v", lo_box_lib9p_msg_as_fmt_formatter(&ctx->ctx.basectx, typ, host_req)); +#pragma GCC diagnostic pop /* Handle it. */ tmessage_handlers[typ](ctx, (void *)host_req, (void *)host_resp); @@ -439,7 +447,11 @@ static void handle_message(struct _lib9p_srv_req *ctx) { typ+1, host_resp, &net_resp)) goto write; +#pragma GCC diagnostic push +#pragma GCC diagnostic ignored "-Wformat" +#pragma GCC diagnostic ignored "-Wformat-extra-args" infof("< %v", lo_box_lib9p_msg_as_fmt_formatter(&ctx->ctx.basectx, typ+1, &host_resp)); +#pragma GCC diagnostic pop write_Rmsg(ctx, &net_resp); } if (host_req) -- cgit v1.2.3-2-g168b From 45f90d1bf6a5f974ed8bc62b6838b5d2e906a3f8 Mon Sep 17 00:00:00 2001 From: "Luke T. Shumaker" Date: Sat, 22 Feb 2025 04:07:17 -0700 Subject: lib9p: srv: Add comments about safety things to do --- lib9p/srv.c | 4 ++++ 1 file changed, 4 insertions(+) (limited to 'lib9p/srv.c') diff --git a/lib9p/srv.c b/lib9p/srv.c index 12556b0..a425dc9 100644 --- a/lib9p/srv.c +++ b/lib9p/srv.c @@ -966,6 +966,8 @@ static void handle_Tread(struct _lib9p_srv_req *ctx, struct lib9p_msg_Rread *resp) { util_handler_common(ctx, req, resp); + /* TODO: serialize simultaneous reads to the same FID */ + /* Check that the FID is valid for this. */ struct _srv_fidinfo *fidinfo = fidmap_load(&ctx->parent_sess->fids, req->fid); if (!fidinfo) { @@ -1028,6 +1030,8 @@ static void handle_Twrite(struct _lib9p_srv_req *ctx, struct lib9p_msg_Rwrite *resp) { util_handler_common(ctx, req, resp); + /* TODO: serialize simultaneous writes to the same FID */ + /* Check that the FID is valid for this. */ struct _srv_fidinfo *fidinfo = fidmap_load(&ctx->parent_sess->fids, req->fid); if (!fidinfo) { -- cgit v1.2.3-2-g168b