Changeset cdd6fc9 in mainline


Ignore:
Timestamp:
2023-01-03T20:38:44Z (16 months ago)
Author:
Jiri Svoboda <jiri@…>
Branches:
master, ticket/834-toolchain-update, topic/msim-upgrade, topic/simplify-dev-export
Children:
3c54869
Parents:
df54aa8
Message:

Add missing replies in IPC error paths

Location:
uspace/lib
Files:
8 edited

Legend:

Unmodified
Added
Removed

  • uspace/lib/ddev/src/ddev_srv.c

    rdf54aa8 rcdd6fc9  
    11/*
    2  * Copyright (c) 2019 Jiri Svoboda
     2 * Copyright (c) 2023 Jiri Svoboda
    33 * All rights reserved.
    44 *
    … …  
    101101        rc = async_data_read_finalize(&call, &info, sizeof(ddev_info_t));
    102102        if (rc != EOK) {
     103                async_answer_0(&call, rc);
    103104                async_answer_0(icall, rc);
    104105                return;

  • uspace/lib/device/src/bd_srv.c

    rdf54aa8 rcdd6fc9  
    11/*
    2  * Copyright (c) 2012 Jiri Svoboda
     2 * Copyright (c) 2023 Jiri Svoboda
    33 * All rights reserved.
    44 *
    … …  
    5656        ipc_call_t rcall;
    5757        if (!async_data_read_receive(&rcall, &size)) {
     58                async_answer_0(&rcall, EINVAL);
    5859                async_answer_0(call, EINVAL);
    5960                return;
    … …  
    99100        ipc_call_t rcall;
    100101        if (!async_data_read_receive(&rcall, &size)) {
     102                async_answer_0(&rcall, EINVAL);
    101103                async_answer_0(call, EINVAL);
    102104                return;

  • uspace/lib/device/src/io/chardev_srv.c

    rdf54aa8 rcdd6fc9  
    11/*
    2  * Copyright (c) 2014 Jiri Svoboda
     2 * Copyright (c) 2023 Jiri Svoboda
    33 * All rights reserved.
    44 *
    … …  
    5555        ipc_call_t call;
    5656        if (!async_data_read_receive(&call, &size)) {
     57                async_answer_0(&call, EINVAL);
    5758                async_answer_0(icall, EINVAL);
    5859                return;

  • uspace/lib/display/src/disp_srv.c

    rdf54aa8 rcdd6fc9  
    11/*
    2  * Copyright (c) 2022 Jiri Svoboda
     2 * Copyright (c) 2023 Jiri Svoboda
    33 * All rights reserved.
    44 *
    … …  
    241241
    242242        if (!async_data_read_receive(&call, &size)) {
     243                async_answer_0(&call, EREFUSED);
    243244                async_answer_0(icall, EREFUSED);
    244245                return;
    … …  
    285286
    286287        if (!async_data_read_receive(&call, &size)) {
     288                async_answer_0(&call, EREFUSED);
    287289                async_answer_0(icall, EREFUSED);
    288290                return;
    … …  
    521523        /* Transfer event data */
    522524        if (!async_data_read_receive(&call, &size)) {
     525                async_answer_0(&call, EREFUSED);
    523526                async_answer_0(icall, EREFUSED);
    524527                return;
    … …  
    555558        /* Transfer information */
    556559        if (!async_data_read_receive(&call, &size)) {
     560                async_answer_0(&call, EREFUSED);
    557561                async_answer_0(icall, EREFUSED);
    558562                return;

  • uspace/lib/inet/src/iplink_srv.c

    rdf54aa8 rcdd6fc9  
    11/*
    2  * Copyright (c) 2021 Jiri Svoboda
     2 * Copyright (c) 2023 Jiri Svoboda
    33 * All rights reserved.
    44 *
    … …  
    7474
    7575        rc = async_data_read_finalize(&call, &mac, size);
    76         if (rc != EOK)
    77                 async_answer_0(&call, rc);
     76        if (rc != EOK) {
     77                async_answer_0(&call, rc);
     78                async_answer_0(icall, rc);
     79                return;
     80        }
    7881
    7982        async_answer_0(icall, rc);
    … …  
    9497        rc = srv->ops->set_mac48(srv, &mac);
    9598        if (rc != EOK) {
     99                async_answer_0(&call, rc);
    96100                async_answer_0(icall, rc);
    97101                return;
    … …  
    99103
    100104        rc = async_data_read_finalize(&call, &mac, sizeof(eth_addr_t));
    101         if (rc != EOK)
    102                 async_answer_0(&call, rc);
     105        if (rc != EOK) {
     106                async_answer_0(&call, rc);
     107                async_answer_0(icall, rc);
     108                return;
     109        }
    103110
    104111        async_answer_0(icall, rc);
    … …  
    109116        ipc_call_t call;
    110117        size_t size;
     118
    111119        if (!async_data_write_receive(&call, &size)) {
    112120                async_answer_0(&call, EREFUSED);
    … …  
    126134                async_answer_0(&call, rc);
    127135                async_answer_0(icall, rc);
     136                return;
    128137        }
    129138
    … …  
    136145        ipc_call_t call;
    137146        size_t size;
     147
    138148        if (!async_data_write_receive(&call, &size)) {
    139149                async_answer_0(&call, EREFUSED);
    … …  
    153163                async_answer_0(&call, rc);
    154164                async_answer_0(icall, rc);
     165                return;
    155166        }
    156167
    … …  
    181192{
    182193        iplink_sdu6_t sdu;
    183 
    184         ipc_call_t call;
    185         size_t size;
     194        ipc_call_t call;
     195        size_t size;
     196
    186197        if (!async_data_write_receive(&call, &size)) {
    187198                async_answer_0(&call, EREFUSED);

  • uspace/lib/ipcgfx/src/server.c

    rdf54aa8 rcdd6fc9  
    11/*
    2  * Copyright (c) 2021 Jiri Svoboda
     2 * Copyright (c) 2023 Jiri Svoboda
    33 * All rights reserved.
    44 *
    … …  
    151151
    152152        if (!async_share_out_receive(&call, &size, &flags)) {
     153                async_answer_0(&call, EINVAL);
    153154                async_answer_0(icall, EINVAL);
    154155                return;
    … …  
    157158        /* Check size */
    158159        if (size != PAGES2SIZE(SIZE2PAGES(dim.x * dim.y * sizeof(uint32_t)))) {
     160                async_answer_0(&call, EINVAL);
    159161                async_answer_0(icall, EINVAL);
    160162                return;
    … …  
    163165        rc = async_share_out_finalize(&call, &pixels);
    164166        if (rc != EOK || pixels == AS_MAP_FAILED) {
     167                async_answer_0(&call, ENOMEM);
    165168                async_answer_0(icall, ENOMEM);
    166169                return;

  • uspace/lib/usbvirt/src/ipc_dev.c

    rdf54aa8 rcdd6fc9  
    11/*
     2 * Copyright (c) 2023 Jiri Svoboda
    23 * Copyright (c) 2011 Vojtech Horky
    34 * All rights reserved.
    … …  
    5960        size_t accepted_size;
    6061        if (!async_data_read_receive(&call, &accepted_size)) {
     62                async_answer_0(&call, EINVAL);
    6163                async_answer_0(icall, EINVAL);
    6264                return;
    … …  
    9496        ipc_call_t data;
    9597        if (!async_data_read_receive(&data, &data_len)) {
     98                async_answer_0(&data, EPARTY);
    9699                async_answer_0(icall, EPARTY);
    97100                free(setup_packet);
    … …  
    101104        void *buffer = malloc(data_len);
    102105        if (buffer == NULL) {
     106                async_answer_0(&data, EPARTY);
    103107                async_answer_0(icall, ENOMEM);
    104108                free(setup_packet);
    … …  
    184188        ipc_call_t data;
    185189        if (!async_data_read_receive(&data, &data_len)) {
     190                async_answer_0(&data, EPARTY);
    186191                async_answer_0(icall, EPARTY);
    187192                return;
    … …  
    190195        void *buffer = malloc(data_len);
    191196        if (buffer == NULL) {
     197                async_answer_0(&data, EPARTY);
    192198                async_answer_0(icall, ENOMEM);
    193199                return;

  • uspace/lib/wndmgt/src/wndmgt_srv.c

    rdf54aa8 rcdd6fc9  
    11/*
    2  * Copyright (c) 2022 Jiri Svoboda
     2 * Copyright (c) 2023 Jiri Svoboda
    33 * All rights reserved.
    44 *
Note: See TracChangeset for help on using the changeset viewer.