source: mainline/kernel/generic/src/ipc/ops/concttome.c@ 5d3ed34

lfn serial ticket/834-toolchain-update topic/msim-upgrade topic/simplify-dev-export
Last change on this file since 5d3ed34 was 5d3ed34, checked in by Jakub Jermar <jakub@…>, 13 years ago

Make sure that both dispatched and non-dispatched calls are properly
answered in ipc_cleanup_call_list().

  • Define simple request_preprocess() hook for IPC_M_CONNECT_TO_ME to detect when the request_process() was not called and no resources allocated so that answer_cleanup() does not attempt to free non-existent resources.
  • Property mode set to 100644
File size: 2.9 KB
Line 
1/*
2 * Copyright (c) 2006 Ondrej Palkovsky
3 * Copyright (c) 2012 Jakub Jermar
4 * All rights reserved.
5 *
6 * Redistribution and use in source and binary forms, with or without
7 * modification, are permitted provided that the following conditions
8 * are met:
9 *
10 * - Redistributions of source code must retain the above copyright
11 * notice, this list of conditions and the following disclaimer.
12 * - Redistributions in binary form must reproduce the above copyright
13 * notice, this list of conditions and the following disclaimer in the
14 * documentation and/or other materials provided with the distribution.
15 * - The name of the author may not be used to endorse or promote products
16 * derived from this software without specific prior written permission.
17 *
18 * THIS SOFTWARE IS PROVIDED BY THE AUTHOR ``AS IS'' AND ANY EXPRESS OR
19 * IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE IMPLIED WARRANTIES
20 * OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE ARE DISCLAIMED.
21 * IN NO EVENT SHALL THE AUTHOR BE LIABLE FOR ANY DIRECT, INDIRECT,
22 * INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT
23 * NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
24 * DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
25 * THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
26 * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF
27 * THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
28 */
29
30/** @addtogroup genericipc
31 * @{
32 */
33/** @file
34 */
35
36#include <ipc/sysipc_ops.h>
37#include <ipc/ipc.h>
38#include <ipc/ipcrsc.h>
39#include <abi/errno.h>
40#include <arch.h>
41
42static int request_preprocess(call_t *call, phone_t *phone)
43{
44 /* Start with the assumption that there is no allocated phoneid. */
45 IPC_SET_ARG5(call->data, -1);
46 return EOK;
47}
48
49static int request_process(call_t *call, answerbox_t *box)
50{
51 int phoneid = phone_alloc(TASK);
52
53 IPC_SET_ARG5(call->data, phoneid);
54
55 return EOK;
56}
57
58static void answer_cleanup(call_t *answer, ipc_data_t *olddata)
59{
60 int phoneid = (int) IPC_GET_ARG5(*olddata);
61
62 if (phoneid >= 0)
63 phone_dealloc(phoneid);
64}
65
66static int answer_preprocess(call_t *answer, ipc_data_t *olddata)
67{
68 int phoneid = (int) IPC_GET_ARG5(*olddata);
69
70 if (IPC_GET_RETVAL(answer->data) != EOK) {
71 /* The connection was not accepted */
72 answer_cleanup(answer, olddata);
73 } else if (phoneid >= 0) {
74 /* The connection was accepted */
75 phone_connect(phoneid, &answer->sender->answerbox);
76 /* Set 'phone hash' as arg5 of response */
77 IPC_SET_ARG5(answer->data, (sysarg_t) &TASK->phones[phoneid]);
78 } else {
79 IPC_SET_RETVAL(answer->data, ELIMIT);
80 }
81
82 return EOK;
83}
84
85
86sysipc_ops_t ipc_m_connect_to_me_ops = {
87 .request_preprocess = request_preprocess,
88 .request_forget = null_request_forget,
89 .request_process = request_process,
90 .answer_cleanup = answer_cleanup,
91 .answer_preprocess = answer_preprocess,
92 .answer_process = null_answer_process,
93};
94
95/** @}
96 */
Note: See TracBrowser for help on using the repository browser.