Mercurial > hg
annotate contrib/chg/hgclient.c @ 45067:9a062913bab6
tests: stablize test-serve.t on Windows
I forget the reason that the subprocess on Windows doesn't print this, but all
other instances of this are similarly conditionalized, so I didn't think too
hard about it.
Also, the server needs to be killed so it doesn't prevent the next run from
working, especially since the port isn't randomized.
Differential Revision: https://phab.mercurial-scm.org/D8720
author | Matt Harbison <matt_harbison@yahoo.com> |
---|---|
date | Thu, 09 Jul 2020 23:03:34 -0400 |
parents | 763b45bc4483 |
children | 0c320e6032f1 |
rev | line source |
---|---|
28060 | 1 /* |
2 * A command server client that uses Unix domain socket | |
3 * | |
4 * Copyright (c) 2011 Yuya Nishihara <yuya@tcha.org> | |
5 * | |
6 * This software may be used and distributed according to the terms of the | |
7 * GNU General Public License version 2 or any later version. | |
8 */ | |
9 | |
35959
9724f54923ec
chg: enable clang-format on all .c and .h files
Augie Fackler <augie@google.com>
parents:
31941
diff
changeset
|
10 #include <arpa/inet.h> /* for ntohl(), htonl() */ |
28060 | 11 #include <assert.h> |
12 #include <ctype.h> | |
13 #include <errno.h> | |
14 #include <fcntl.h> | |
15 #include <signal.h> | |
16 #include <stdint.h> | |
17 #include <stdio.h> | |
18 #include <stdlib.h> | |
19 #include <string.h> | |
20 #include <sys/socket.h> | |
21 #include <sys/stat.h> | |
22 #include <sys/un.h> | |
23 #include <unistd.h> | |
24 | |
25 #include "hgclient.h" | |
30738
a45c0f42271f
chg: handle pager request client-side
Jun Wu <quark@fb.com>
parents:
30728
diff
changeset
|
26 #include "procutil.h" |
28060 | 27 #include "util.h" |
28 | |
35959
9724f54923ec
chg: enable clang-format on all .c and .h files
Augie Fackler <augie@google.com>
parents:
31941
diff
changeset
|
29 enum { CAP_GETENCODING = 0x0001, |
9724f54923ec
chg: enable clang-format on all .c and .h files
Augie Fackler <augie@google.com>
parents:
31941
diff
changeset
|
30 CAP_RUNCOMMAND = 0x0002, |
9724f54923ec
chg: enable clang-format on all .c and .h files
Augie Fackler <augie@google.com>
parents:
31941
diff
changeset
|
31 /* cHg extension: */ |
9724f54923ec
chg: enable clang-format on all .c and .h files
Augie Fackler <augie@google.com>
parents:
31941
diff
changeset
|
32 CAP_ATTACHIO = 0x0100, |
9724f54923ec
chg: enable clang-format on all .c and .h files
Augie Fackler <augie@google.com>
parents:
31941
diff
changeset
|
33 CAP_CHDIR = 0x0200, |
9724f54923ec
chg: enable clang-format on all .c and .h files
Augie Fackler <augie@google.com>
parents:
31941
diff
changeset
|
34 CAP_SETENV = 0x0800, |
40109
413b6b10fdd5
chg: upgrade client to use "setumask2" command
Yuya Nishihara <yuya@tcha.org>
parents:
35959
diff
changeset
|
35 CAP_SETUMASK2 = 0x1000, |
35959
9724f54923ec
chg: enable clang-format on all .c and .h files
Augie Fackler <augie@google.com>
parents:
31941
diff
changeset
|
36 CAP_VALIDATE = 0x2000, |
9724f54923ec
chg: enable clang-format on all .c and .h files
Augie Fackler <augie@google.com>
parents:
31941
diff
changeset
|
37 CAP_SETPROCNAME = 0x4000, |
28060 | 38 }; |
39 | |
40 typedef struct { | |
41 const char *name; | |
42 unsigned int flag; | |
43 } cappair_t; | |
44 | |
45 static const cappair_t captable[] = { | |
35959
9724f54923ec
chg: enable clang-format on all .c and .h files
Augie Fackler <augie@google.com>
parents:
31941
diff
changeset
|
46 {"getencoding", CAP_GETENCODING}, |
9724f54923ec
chg: enable clang-format on all .c and .h files
Augie Fackler <augie@google.com>
parents:
31941
diff
changeset
|
47 {"runcommand", CAP_RUNCOMMAND}, |
9724f54923ec
chg: enable clang-format on all .c and .h files
Augie Fackler <augie@google.com>
parents:
31941
diff
changeset
|
48 {"attachio", CAP_ATTACHIO}, |
9724f54923ec
chg: enable clang-format on all .c and .h files
Augie Fackler <augie@google.com>
parents:
31941
diff
changeset
|
49 {"chdir", CAP_CHDIR}, |
9724f54923ec
chg: enable clang-format on all .c and .h files
Augie Fackler <augie@google.com>
parents:
31941
diff
changeset
|
50 {"setenv", CAP_SETENV}, |
40109
413b6b10fdd5
chg: upgrade client to use "setumask2" command
Yuya Nishihara <yuya@tcha.org>
parents:
35959
diff
changeset
|
51 {"setumask2", CAP_SETUMASK2}, |
35959
9724f54923ec
chg: enable clang-format on all .c and .h files
Augie Fackler <augie@google.com>
parents:
31941
diff
changeset
|
52 {"validate", CAP_VALIDATE}, |
9724f54923ec
chg: enable clang-format on all .c and .h files
Augie Fackler <augie@google.com>
parents:
31941
diff
changeset
|
53 {"setprocname", CAP_SETPROCNAME}, |
9724f54923ec
chg: enable clang-format on all .c and .h files
Augie Fackler <augie@google.com>
parents:
31941
diff
changeset
|
54 {NULL, 0}, /* terminator */ |
28060 | 55 }; |
56 | |
57 typedef struct { | |
58 char ch; | |
59 char *data; | |
60 size_t maxdatasize; | |
61 size_t datasize; | |
62 } context_t; | |
63 | |
64 struct hgclient_tag_ { | |
65 int sockfd; | |
29581 | 66 pid_t pgid; |
28060 | 67 pid_t pid; |
68 context_t ctx; | |
69 unsigned int capflags; | |
70 }; | |
71 | |
72 static const size_t defaultdatasize = 4096; | |
73 | |
30738
a45c0f42271f
chg: handle pager request client-side
Jun Wu <quark@fb.com>
parents:
30728
diff
changeset
|
74 static void attachio(hgclient_t *hgc); |
a45c0f42271f
chg: handle pager request client-side
Jun Wu <quark@fb.com>
parents:
30728
diff
changeset
|
75 |
28060 | 76 static void initcontext(context_t *ctx) |
77 { | |
78 ctx->ch = '\0'; | |
79 ctx->data = malloc(defaultdatasize); | |
80 ctx->maxdatasize = (ctx->data) ? defaultdatasize : 0; | |
81 ctx->datasize = 0; | |
82 debugmsg("initialize context buffer with size %zu", ctx->maxdatasize); | |
83 } | |
84 | |
85 static void enlargecontext(context_t *ctx, size_t newsize) | |
86 { | |
41336
763b45bc4483
cleanup: use clang-tidy to add missing {} around one-line statements
Augie Fackler <augie@google.com>
parents:
40109
diff
changeset
|
87 if (newsize <= ctx->maxdatasize) { |
28060 | 88 return; |
41336
763b45bc4483
cleanup: use clang-tidy to add missing {} around one-line statements
Augie Fackler <augie@google.com>
parents:
40109
diff
changeset
|
89 } |
28060 | 90 |
35959
9724f54923ec
chg: enable clang-format on all .c and .h files
Augie Fackler <augie@google.com>
parents:
31941
diff
changeset
|
91 newsize = defaultdatasize * |
9724f54923ec
chg: enable clang-format on all .c and .h files
Augie Fackler <augie@google.com>
parents:
31941
diff
changeset
|
92 ((newsize + defaultdatasize - 1) / defaultdatasize); |
28166
c6e0c2533e5f
chg: use mallocx and reallocx in hgclient
Jun Wu <quark@fb.com>
parents:
28160
diff
changeset
|
93 ctx->data = reallocx(ctx->data, newsize); |
28060 | 94 ctx->maxdatasize = newsize; |
95 debugmsg("enlarge context buffer to %zu", ctx->maxdatasize); | |
96 } | |
97 | |
98 static void freecontext(context_t *ctx) | |
99 { | |
100 debugmsg("free context buffer"); | |
101 free(ctx->data); | |
102 ctx->data = NULL; | |
103 ctx->maxdatasize = 0; | |
104 ctx->datasize = 0; | |
105 } | |
106 | |
107 /* Read channeled response from cmdserver */ | |
108 static void readchannel(hgclient_t *hgc) | |
109 { | |
110 assert(hgc); | |
111 | |
112 ssize_t rsize = recv(hgc->sockfd, &hgc->ctx.ch, sizeof(hgc->ctx.ch), 0); | |
28551
8e5312f8df30
chg: downgrade "failed to read channel" from abortmsg to debugmsg
Jun Wu <quark@fb.com>
parents:
28535
diff
changeset
|
113 if (rsize != sizeof(hgc->ctx.ch)) { |
8e5312f8df30
chg: downgrade "failed to read channel" from abortmsg to debugmsg
Jun Wu <quark@fb.com>
parents:
28535
diff
changeset
|
114 /* server would have exception and traceback would be printed */ |
8e5312f8df30
chg: downgrade "failed to read channel" from abortmsg to debugmsg
Jun Wu <quark@fb.com>
parents:
28535
diff
changeset
|
115 debugmsg("failed to read channel"); |
8e5312f8df30
chg: downgrade "failed to read channel" from abortmsg to debugmsg
Jun Wu <quark@fb.com>
parents:
28535
diff
changeset
|
116 exit(255); |
8e5312f8df30
chg: downgrade "failed to read channel" from abortmsg to debugmsg
Jun Wu <quark@fb.com>
parents:
28535
diff
changeset
|
117 } |
28060 | 118 |
119 uint32_t datasize_n; | |
120 rsize = recv(hgc->sockfd, &datasize_n, sizeof(datasize_n), 0); | |
41336
763b45bc4483
cleanup: use clang-tidy to add missing {} around one-line statements
Augie Fackler <augie@google.com>
parents:
40109
diff
changeset
|
121 if (rsize != sizeof(datasize_n)) { |
28060 | 122 abortmsg("failed to read data size"); |
41336
763b45bc4483
cleanup: use clang-tidy to add missing {} around one-line statements
Augie Fackler <augie@google.com>
parents:
40109
diff
changeset
|
123 } |
28060 | 124 |
125 /* datasize denotes the maximum size to write if input request */ | |
126 hgc->ctx.datasize = ntohl(datasize_n); | |
127 enlargecontext(&hgc->ctx, hgc->ctx.datasize); | |
128 | |
41336
763b45bc4483
cleanup: use clang-tidy to add missing {} around one-line statements
Augie Fackler <augie@google.com>
parents:
40109
diff
changeset
|
129 if (isupper(hgc->ctx.ch) && hgc->ctx.ch != 'S') { |
35959
9724f54923ec
chg: enable clang-format on all .c and .h files
Augie Fackler <augie@google.com>
parents:
31941
diff
changeset
|
130 return; /* assumes input request */ |
41336
763b45bc4483
cleanup: use clang-tidy to add missing {} around one-line statements
Augie Fackler <augie@google.com>
parents:
40109
diff
changeset
|
131 } |
28060 | 132 |
133 size_t cursize = 0; | |
134 while (cursize < hgc->ctx.datasize) { | |
135 rsize = recv(hgc->sockfd, hgc->ctx.data + cursize, | |
35959
9724f54923ec
chg: enable clang-format on all .c and .h files
Augie Fackler <augie@google.com>
parents:
31941
diff
changeset
|
136 hgc->ctx.datasize - cursize, 0); |
41336
763b45bc4483
cleanup: use clang-tidy to add missing {} around one-line statements
Augie Fackler <augie@google.com>
parents:
40109
diff
changeset
|
137 if (rsize < 1) { |
28060 | 138 abortmsg("failed to read data block"); |
41336
763b45bc4483
cleanup: use clang-tidy to add missing {} around one-line statements
Augie Fackler <augie@google.com>
parents:
40109
diff
changeset
|
139 } |
28060 | 140 cursize += rsize; |
141 } | |
142 } | |
143 | |
144 static void sendall(int sockfd, const void *data, size_t datasize) | |
145 { | |
146 const char *p = data; | |
147 const char *const endp = p + datasize; | |
148 while (p < endp) { | |
149 ssize_t r = send(sockfd, p, endp - p, 0); | |
41336
763b45bc4483
cleanup: use clang-tidy to add missing {} around one-line statements
Augie Fackler <augie@google.com>
parents:
40109
diff
changeset
|
150 if (r < 0) { |
28789
7f6e0a15189b
chg: replace abortmsg showing errno with abortmsgerrno
Jun Wu <quark@fb.com>
parents:
28769
diff
changeset
|
151 abortmsgerrno("cannot communicate"); |
41336
763b45bc4483
cleanup: use clang-tidy to add missing {} around one-line statements
Augie Fackler <augie@google.com>
parents:
40109
diff
changeset
|
152 } |
28060 | 153 p += r; |
154 } | |
155 } | |
156 | |
157 /* Write lengh-data block to cmdserver */ | |
158 static void writeblock(const hgclient_t *hgc) | |
159 { | |
160 assert(hgc); | |
161 | |
162 const uint32_t datasize_n = htonl(hgc->ctx.datasize); | |
163 sendall(hgc->sockfd, &datasize_n, sizeof(datasize_n)); | |
164 | |
165 sendall(hgc->sockfd, hgc->ctx.data, hgc->ctx.datasize); | |
166 } | |
167 | |
168 static void writeblockrequest(const hgclient_t *hgc, const char *chcmd) | |
169 { | |
170 debugmsg("request %s, block size %zu", chcmd, hgc->ctx.datasize); | |
171 | |
172 char buf[strlen(chcmd) + 1]; | |
173 memcpy(buf, chcmd, sizeof(buf) - 1); | |
174 buf[sizeof(buf) - 1] = '\n'; | |
175 sendall(hgc->sockfd, buf, sizeof(buf)); | |
176 | |
177 writeblock(hgc); | |
178 } | |
179 | |
180 /* Build '\0'-separated list of args. argsize < 0 denotes that args are | |
181 * terminated by NULL. */ | |
182 static void packcmdargs(context_t *ctx, const char *const args[], | |
35959
9724f54923ec
chg: enable clang-format on all .c and .h files
Augie Fackler <augie@google.com>
parents:
31941
diff
changeset
|
183 ssize_t argsize) |
28060 | 184 { |
185 ctx->datasize = 0; | |
186 const char *const *const end = (argsize >= 0) ? args + argsize : NULL; | |
187 for (const char *const *it = args; it != end && *it; ++it) { | |
35959
9724f54923ec
chg: enable clang-format on all .c and .h files
Augie Fackler <augie@google.com>
parents:
31941
diff
changeset
|
188 const size_t n = strlen(*it) + 1; /* include '\0' */ |
28060 | 189 enlargecontext(ctx, ctx->datasize + n); |
190 memcpy(ctx->data + ctx->datasize, *it, n); | |
191 ctx->datasize += n; | |
192 } | |
193 | |
41336
763b45bc4483
cleanup: use clang-tidy to add missing {} around one-line statements
Augie Fackler <augie@google.com>
parents:
40109
diff
changeset
|
194 if (ctx->datasize > 0) { |
35959
9724f54923ec
chg: enable clang-format on all .c and .h files
Augie Fackler <augie@google.com>
parents:
31941
diff
changeset
|
195 --ctx->datasize; /* strip last '\0' */ |
41336
763b45bc4483
cleanup: use clang-tidy to add missing {} around one-line statements
Augie Fackler <augie@google.com>
parents:
40109
diff
changeset
|
196 } |
28060 | 197 } |
198 | |
199 /* Extract '\0'-separated list of args to new buffer, terminated by NULL */ | |
200 static const char **unpackcmdargsnul(const context_t *ctx) | |
201 { | |
202 const char **args = NULL; | |
203 size_t nargs = 0, maxnargs = 0; | |
204 const char *s = ctx->data; | |
205 const char *e = ctx->data + ctx->datasize; | |
206 for (;;) { | |
35959
9724f54923ec
chg: enable clang-format on all .c and .h files
Augie Fackler <augie@google.com>
parents:
31941
diff
changeset
|
207 if (nargs + 1 >= maxnargs) { /* including last NULL */ |
28060 | 208 maxnargs += 256; |
28166
c6e0c2533e5f
chg: use mallocx and reallocx in hgclient
Jun Wu <quark@fb.com>
parents:
28160
diff
changeset
|
209 args = reallocx(args, maxnargs * sizeof(args[0])); |
28060 | 210 } |
211 args[nargs] = s; | |
212 nargs++; | |
213 s = memchr(s, '\0', e - s); | |
41336
763b45bc4483
cleanup: use clang-tidy to add missing {} around one-line statements
Augie Fackler <augie@google.com>
parents:
40109
diff
changeset
|
214 if (!s) { |
28060 | 215 break; |
41336
763b45bc4483
cleanup: use clang-tidy to add missing {} around one-line statements
Augie Fackler <augie@google.com>
parents:
40109
diff
changeset
|
216 } |
28060 | 217 s++; |
218 } | |
219 args[nargs] = NULL; | |
220 return args; | |
221 } | |
222 | |
223 static void handlereadrequest(hgclient_t *hgc) | |
224 { | |
225 context_t *ctx = &hgc->ctx; | |
226 size_t r = fread(ctx->data, sizeof(ctx->data[0]), ctx->datasize, stdin); | |
227 ctx->datasize = r; | |
228 writeblock(hgc); | |
229 } | |
230 | |
231 /* Read single-line */ | |
232 static void handlereadlinerequest(hgclient_t *hgc) | |
233 { | |
234 context_t *ctx = &hgc->ctx; | |
41336
763b45bc4483
cleanup: use clang-tidy to add missing {} around one-line statements
Augie Fackler <augie@google.com>
parents:
40109
diff
changeset
|
235 if (!fgets(ctx->data, ctx->datasize, stdin)) { |
28060 | 236 ctx->data[0] = '\0'; |
41336
763b45bc4483
cleanup: use clang-tidy to add missing {} around one-line statements
Augie Fackler <augie@google.com>
parents:
40109
diff
changeset
|
237 } |
28060 | 238 ctx->datasize = strlen(ctx->data); |
239 writeblock(hgc); | |
240 } | |
241 | |
242 /* Execute the requested command and write exit code */ | |
243 static void handlesystemrequest(hgclient_t *hgc) | |
244 { | |
245 context_t *ctx = &hgc->ctx; | |
246 enlargecontext(ctx, ctx->datasize + 1); | |
35959
9724f54923ec
chg: enable clang-format on all .c and .h files
Augie Fackler <augie@google.com>
parents:
31941
diff
changeset
|
247 ctx->data[ctx->datasize] = '\0'; /* terminate last string */ |
28060 | 248 |
249 const char **args = unpackcmdargsnul(ctx); | |
41336
763b45bc4483
cleanup: use clang-tidy to add missing {} around one-line statements
Augie Fackler <augie@google.com>
parents:
40109
diff
changeset
|
250 if (!args[0] || !args[1] || !args[2]) { |
30726
dd897eb1699e
chg: send type information via S channel (BC)
Jun Wu <quark@fb.com>
parents:
30679
diff
changeset
|
251 abortmsg("missing type or command or cwd in system request"); |
41336
763b45bc4483
cleanup: use clang-tidy to add missing {} around one-line statements
Augie Fackler <augie@google.com>
parents:
40109
diff
changeset
|
252 } |
30728
7438cb35979a
chg: check type read from S channel
Jun Wu <quark@fb.com>
parents:
30726
diff
changeset
|
253 if (strcmp(args[0], "system") == 0) { |
7438cb35979a
chg: check type read from S channel
Jun Wu <quark@fb.com>
parents:
30726
diff
changeset
|
254 debugmsg("run '%s' at '%s'", args[1], args[2]); |
7438cb35979a
chg: check type read from S channel
Jun Wu <quark@fb.com>
parents:
30726
diff
changeset
|
255 int32_t r = runshellcmd(args[1], args + 3, args[2]); |
7438cb35979a
chg: check type read from S channel
Jun Wu <quark@fb.com>
parents:
30726
diff
changeset
|
256 free(args); |
28060 | 257 |
30728
7438cb35979a
chg: check type read from S channel
Jun Wu <quark@fb.com>
parents:
30726
diff
changeset
|
258 uint32_t r_n = htonl(r); |
7438cb35979a
chg: check type read from S channel
Jun Wu <quark@fb.com>
parents:
30726
diff
changeset
|
259 memcpy(ctx->data, &r_n, sizeof(r_n)); |
7438cb35979a
chg: check type read from S channel
Jun Wu <quark@fb.com>
parents:
30726
diff
changeset
|
260 ctx->datasize = sizeof(r_n); |
7438cb35979a
chg: check type read from S channel
Jun Wu <quark@fb.com>
parents:
30726
diff
changeset
|
261 writeblock(hgc); |
30738
a45c0f42271f
chg: handle pager request client-side
Jun Wu <quark@fb.com>
parents:
30728
diff
changeset
|
262 } else if (strcmp(args[0], "pager") == 0) { |
31941
ac5527021097
chg: respect environment variables for pager
Jun Wu <quark@fb.com>
parents:
30756
diff
changeset
|
263 setuppager(args[1], args + 3); |
41336
763b45bc4483
cleanup: use clang-tidy to add missing {} around one-line statements
Augie Fackler <augie@google.com>
parents:
40109
diff
changeset
|
264 if (hgc->capflags & CAP_ATTACHIO) { |
30738
a45c0f42271f
chg: handle pager request client-side
Jun Wu <quark@fb.com>
parents:
30728
diff
changeset
|
265 attachio(hgc); |
41336
763b45bc4483
cleanup: use clang-tidy to add missing {} around one-line statements
Augie Fackler <augie@google.com>
parents:
40109
diff
changeset
|
266 } |
30738
a45c0f42271f
chg: handle pager request client-side
Jun Wu <quark@fb.com>
parents:
30728
diff
changeset
|
267 /* unblock the server */ |
a45c0f42271f
chg: handle pager request client-side
Jun Wu <quark@fb.com>
parents:
30728
diff
changeset
|
268 static const char emptycmd[] = "\n"; |
a45c0f42271f
chg: handle pager request client-side
Jun Wu <quark@fb.com>
parents:
30728
diff
changeset
|
269 sendall(hgc->sockfd, emptycmd, sizeof(emptycmd) - 1); |
30728
7438cb35979a
chg: check type read from S channel
Jun Wu <quark@fb.com>
parents:
30726
diff
changeset
|
270 } else { |
7438cb35979a
chg: check type read from S channel
Jun Wu <quark@fb.com>
parents:
30726
diff
changeset
|
271 abortmsg("unknown type in system request: %s", args[0]); |
7438cb35979a
chg: check type read from S channel
Jun Wu <quark@fb.com>
parents:
30726
diff
changeset
|
272 } |
28060 | 273 } |
274 | |
275 /* Read response of command execution until receiving 'r'-esult */ | |
276 static void handleresponse(hgclient_t *hgc) | |
277 { | |
278 for (;;) { | |
279 readchannel(hgc); | |
280 context_t *ctx = &hgc->ctx; | |
35959
9724f54923ec
chg: enable clang-format on all .c and .h files
Augie Fackler <augie@google.com>
parents:
31941
diff
changeset
|
281 debugmsg("response read from channel %c, size %zu", ctx->ch, |
9724f54923ec
chg: enable clang-format on all .c and .h files
Augie Fackler <augie@google.com>
parents:
31941
diff
changeset
|
282 ctx->datasize); |
28060 | 283 switch (ctx->ch) { |
284 case 'o': | |
285 fwrite(ctx->data, sizeof(ctx->data[0]), ctx->datasize, | |
286 stdout); | |
287 break; | |
288 case 'e': | |
289 fwrite(ctx->data, sizeof(ctx->data[0]), ctx->datasize, | |
290 stderr); | |
291 break; | |
292 case 'd': | |
293 /* assumes last char is '\n' */ | |
294 ctx->data[ctx->datasize - 1] = '\0'; | |
295 debugmsg("server: %s", ctx->data); | |
296 break; | |
297 case 'r': | |
298 return; | |
299 case 'I': | |
300 handlereadrequest(hgc); | |
301 break; | |
302 case 'L': | |
303 handlereadlinerequest(hgc); | |
304 break; | |
305 case 'S': | |
306 handlesystemrequest(hgc); | |
307 break; | |
308 default: | |
41336
763b45bc4483
cleanup: use clang-tidy to add missing {} around one-line statements
Augie Fackler <augie@google.com>
parents:
40109
diff
changeset
|
309 if (isupper(ctx->ch)) { |
28060 | 310 abortmsg("cannot handle response (ch = %c)", |
35959
9724f54923ec
chg: enable clang-format on all .c and .h files
Augie Fackler <augie@google.com>
parents:
31941
diff
changeset
|
311 ctx->ch); |
41336
763b45bc4483
cleanup: use clang-tidy to add missing {} around one-line statements
Augie Fackler <augie@google.com>
parents:
40109
diff
changeset
|
312 } |
28060 | 313 } |
314 } | |
315 } | |
316 | |
317 static unsigned int parsecapabilities(const char *s, const char *e) | |
318 { | |
319 unsigned int flags = 0; | |
320 while (s < e) { | |
321 const char *t = strchr(s, ' '); | |
41336
763b45bc4483
cleanup: use clang-tidy to add missing {} around one-line statements
Augie Fackler <augie@google.com>
parents:
40109
diff
changeset
|
322 if (!t || t > e) { |
28060 | 323 t = e; |
41336
763b45bc4483
cleanup: use clang-tidy to add missing {} around one-line statements
Augie Fackler <augie@google.com>
parents:
40109
diff
changeset
|
324 } |
28060 | 325 const cappair_t *cap; |
326 for (cap = captable; cap->flag; ++cap) { | |
327 size_t n = t - s; | |
328 if (strncmp(s, cap->name, n) == 0 && | |
329 strlen(cap->name) == n) { | |
330 flags |= cap->flag; | |
331 break; | |
332 } | |
333 } | |
334 s = t + 1; | |
335 } | |
336 return flags; | |
337 } | |
338 | |
339 static void readhello(hgclient_t *hgc) | |
340 { | |
341 readchannel(hgc); | |
342 context_t *ctx = &hgc->ctx; | |
28512
b957b4c6cad8
chg: provide early exception to user
Yuya Nishihara <yuya@tcha.org>
parents:
28356
diff
changeset
|
343 if (ctx->ch != 'o') { |
b957b4c6cad8
chg: provide early exception to user
Yuya Nishihara <yuya@tcha.org>
parents:
28356
diff
changeset
|
344 char ch = ctx->ch; |
b957b4c6cad8
chg: provide early exception to user
Yuya Nishihara <yuya@tcha.org>
parents:
28356
diff
changeset
|
345 if (ch == 'e') { |
b957b4c6cad8
chg: provide early exception to user
Yuya Nishihara <yuya@tcha.org>
parents:
28356
diff
changeset
|
346 /* write early error and will exit */ |
b957b4c6cad8
chg: provide early exception to user
Yuya Nishihara <yuya@tcha.org>
parents:
28356
diff
changeset
|
347 fwrite(ctx->data, sizeof(ctx->data[0]), ctx->datasize, |
b957b4c6cad8
chg: provide early exception to user
Yuya Nishihara <yuya@tcha.org>
parents:
28356
diff
changeset
|
348 stderr); |
b957b4c6cad8
chg: provide early exception to user
Yuya Nishihara <yuya@tcha.org>
parents:
28356
diff
changeset
|
349 handleresponse(hgc); |
b957b4c6cad8
chg: provide early exception to user
Yuya Nishihara <yuya@tcha.org>
parents:
28356
diff
changeset
|
350 } |
b957b4c6cad8
chg: provide early exception to user
Yuya Nishihara <yuya@tcha.org>
parents:
28356
diff
changeset
|
351 abortmsg("unexpected channel of hello message (ch = %c)", ch); |
b957b4c6cad8
chg: provide early exception to user
Yuya Nishihara <yuya@tcha.org>
parents:
28356
diff
changeset
|
352 } |
28060 | 353 enlargecontext(ctx, ctx->datasize + 1); |
354 ctx->data[ctx->datasize] = '\0'; | |
355 debugmsg("hello received: %s (size = %zu)", ctx->data, ctx->datasize); | |
356 | |
357 const char *s = ctx->data; | |
358 const char *const dataend = ctx->data + ctx->datasize; | |
359 while (s < dataend) { | |
360 const char *t = strchr(s, ':'); | |
41336
763b45bc4483
cleanup: use clang-tidy to add missing {} around one-line statements
Augie Fackler <augie@google.com>
parents:
40109
diff
changeset
|
361 if (!t || t[1] != ' ') { |
28060 | 362 break; |
41336
763b45bc4483
cleanup: use clang-tidy to add missing {} around one-line statements
Augie Fackler <augie@google.com>
parents:
40109
diff
changeset
|
363 } |
28060 | 364 const char *u = strchr(t + 2, '\n'); |
41336
763b45bc4483
cleanup: use clang-tidy to add missing {} around one-line statements
Augie Fackler <augie@google.com>
parents:
40109
diff
changeset
|
365 if (!u) { |
28060 | 366 u = dataend; |
41336
763b45bc4483
cleanup: use clang-tidy to add missing {} around one-line statements
Augie Fackler <augie@google.com>
parents:
40109
diff
changeset
|
367 } |
28060 | 368 if (strncmp(s, "capabilities:", t - s + 1) == 0) { |
369 hgc->capflags = parsecapabilities(t + 2, u); | |
29581 | 370 } else if (strncmp(s, "pgid:", t - s + 1) == 0) { |
371 hgc->pgid = strtol(t + 2, NULL, 10); | |
28060 | 372 } else if (strncmp(s, "pid:", t - s + 1) == 0) { |
373 hgc->pid = strtol(t + 2, NULL, 10); | |
374 } | |
375 s = u + 1; | |
376 } | |
377 debugmsg("capflags=0x%04x, pid=%d", hgc->capflags, hgc->pid); | |
378 } | |
379 | |
30751 | 380 static void updateprocname(hgclient_t *hgc) |
381 { | |
35959
9724f54923ec
chg: enable clang-format on all .c and .h files
Augie Fackler <augie@google.com>
parents:
31941
diff
changeset
|
382 int r = snprintf(hgc->ctx.data, hgc->ctx.maxdatasize, "chg[worker/%d]", |
9724f54923ec
chg: enable clang-format on all .c and .h files
Augie Fackler <augie@google.com>
parents:
31941
diff
changeset
|
383 (int)getpid()); |
41336
763b45bc4483
cleanup: use clang-tidy to add missing {} around one-line statements
Augie Fackler <augie@google.com>
parents:
40109
diff
changeset
|
384 if (r < 0 || (size_t)r >= hgc->ctx.maxdatasize) { |
30756
1f9684fe94cc
chg: check snprintf result strictly
Jun Wu <quark@fb.com>
parents:
30751
diff
changeset
|
385 abortmsg("insufficient buffer to write procname (r = %d)", r); |
41336
763b45bc4483
cleanup: use clang-tidy to add missing {} around one-line statements
Augie Fackler <augie@google.com>
parents:
40109
diff
changeset
|
386 } |
30756
1f9684fe94cc
chg: check snprintf result strictly
Jun Wu <quark@fb.com>
parents:
30751
diff
changeset
|
387 hgc->ctx.datasize = (size_t)r; |
30751 | 388 writeblockrequest(hgc, "setprocname"); |
389 } | |
390 | |
28060 | 391 static void attachio(hgclient_t *hgc) |
392 { | |
393 debugmsg("request attachio"); | |
394 static const char chcmd[] = "attachio\n"; | |
395 sendall(hgc->sockfd, chcmd, sizeof(chcmd) - 1); | |
396 readchannel(hgc); | |
397 context_t *ctx = &hgc->ctx; | |
41336
763b45bc4483
cleanup: use clang-tidy to add missing {} around one-line statements
Augie Fackler <augie@google.com>
parents:
40109
diff
changeset
|
398 if (ctx->ch != 'I') { |
28060 | 399 abortmsg("unexpected response for attachio (ch = %c)", ctx->ch); |
41336
763b45bc4483
cleanup: use clang-tidy to add missing {} around one-line statements
Augie Fackler <augie@google.com>
parents:
40109
diff
changeset
|
400 } |
28060 | 401 |
402 static const int fds[3] = {STDIN_FILENO, STDOUT_FILENO, STDERR_FILENO}; | |
403 struct msghdr msgh; | |
404 memset(&msgh, 0, sizeof(msgh)); | |
35959
9724f54923ec
chg: enable clang-format on all .c and .h files
Augie Fackler <augie@google.com>
parents:
31941
diff
changeset
|
405 struct iovec iov = {ctx->data, ctx->datasize}; /* dummy payload */ |
28060 | 406 msgh.msg_iov = &iov; |
407 msgh.msg_iovlen = 1; | |
408 char fdbuf[CMSG_SPACE(sizeof(fds))]; | |
409 msgh.msg_control = fdbuf; | |
410 msgh.msg_controllen = sizeof(fdbuf); | |
411 struct cmsghdr *cmsg = CMSG_FIRSTHDR(&msgh); | |
412 cmsg->cmsg_level = SOL_SOCKET; | |
413 cmsg->cmsg_type = SCM_RIGHTS; | |
414 cmsg->cmsg_len = CMSG_LEN(sizeof(fds)); | |
415 memcpy(CMSG_DATA(cmsg), fds, sizeof(fds)); | |
416 msgh.msg_controllen = cmsg->cmsg_len; | |
417 ssize_t r = sendmsg(hgc->sockfd, &msgh, 0); | |
41336
763b45bc4483
cleanup: use clang-tidy to add missing {} around one-line statements
Augie Fackler <augie@google.com>
parents:
40109
diff
changeset
|
418 if (r < 0) { |
28789
7f6e0a15189b
chg: replace abortmsg showing errno with abortmsgerrno
Jun Wu <quark@fb.com>
parents:
28769
diff
changeset
|
419 abortmsgerrno("sendmsg failed"); |
41336
763b45bc4483
cleanup: use clang-tidy to add missing {} around one-line statements
Augie Fackler <augie@google.com>
parents:
40109
diff
changeset
|
420 } |
28060 | 421 |
422 handleresponse(hgc); | |
423 int32_t n; | |
41336
763b45bc4483
cleanup: use clang-tidy to add missing {} around one-line statements
Augie Fackler <augie@google.com>
parents:
40109
diff
changeset
|
424 if (ctx->datasize != sizeof(n)) { |
28060 | 425 abortmsg("unexpected size of attachio result"); |
41336
763b45bc4483
cleanup: use clang-tidy to add missing {} around one-line statements
Augie Fackler <augie@google.com>
parents:
40109
diff
changeset
|
426 } |
28060 | 427 memcpy(&n, ctx->data, sizeof(n)); |
428 n = ntohl(n); | |
41336
763b45bc4483
cleanup: use clang-tidy to add missing {} around one-line statements
Augie Fackler <augie@google.com>
parents:
40109
diff
changeset
|
429 if (n != sizeof(fds) / sizeof(fds[0])) { |
28060 | 430 abortmsg("failed to send fds (n = %d)", n); |
41336
763b45bc4483
cleanup: use clang-tidy to add missing {} around one-line statements
Augie Fackler <augie@google.com>
parents:
40109
diff
changeset
|
431 } |
28060 | 432 } |
433 | |
434 static void chdirtocwd(hgclient_t *hgc) | |
435 { | |
41336
763b45bc4483
cleanup: use clang-tidy to add missing {} around one-line statements
Augie Fackler <augie@google.com>
parents:
40109
diff
changeset
|
436 if (!getcwd(hgc->ctx.data, hgc->ctx.maxdatasize)) { |
28789
7f6e0a15189b
chg: replace abortmsg showing errno with abortmsgerrno
Jun Wu <quark@fb.com>
parents:
28769
diff
changeset
|
437 abortmsgerrno("failed to getcwd"); |
41336
763b45bc4483
cleanup: use clang-tidy to add missing {} around one-line statements
Augie Fackler <augie@google.com>
parents:
40109
diff
changeset
|
438 } |
28060 | 439 hgc->ctx.datasize = strlen(hgc->ctx.data); |
440 writeblockrequest(hgc, "chdir"); | |
441 } | |
442 | |
28160
098cb7bd46a7
chg: forward umask from client to server
Jun Wu <quark@fb.com>
parents:
28060
diff
changeset
|
443 static void forwardumask(hgclient_t *hgc) |
098cb7bd46a7
chg: forward umask from client to server
Jun Wu <quark@fb.com>
parents:
28060
diff
changeset
|
444 { |
098cb7bd46a7
chg: forward umask from client to server
Jun Wu <quark@fb.com>
parents:
28060
diff
changeset
|
445 mode_t mask = umask(0); |
098cb7bd46a7
chg: forward umask from client to server
Jun Wu <quark@fb.com>
parents:
28060
diff
changeset
|
446 umask(mask); |
098cb7bd46a7
chg: forward umask from client to server
Jun Wu <quark@fb.com>
parents:
28060
diff
changeset
|
447 |
098cb7bd46a7
chg: forward umask from client to server
Jun Wu <quark@fb.com>
parents:
28060
diff
changeset
|
448 uint32_t data = htonl(mask); |
40109
413b6b10fdd5
chg: upgrade client to use "setumask2" command
Yuya Nishihara <yuya@tcha.org>
parents:
35959
diff
changeset
|
449 enlargecontext(&hgc->ctx, sizeof(data)); |
413b6b10fdd5
chg: upgrade client to use "setumask2" command
Yuya Nishihara <yuya@tcha.org>
parents:
35959
diff
changeset
|
450 memcpy(hgc->ctx.data, &data, sizeof(data)); |
413b6b10fdd5
chg: upgrade client to use "setumask2" command
Yuya Nishihara <yuya@tcha.org>
parents:
35959
diff
changeset
|
451 hgc->ctx.datasize = sizeof(data); |
413b6b10fdd5
chg: upgrade client to use "setumask2" command
Yuya Nishihara <yuya@tcha.org>
parents:
35959
diff
changeset
|
452 writeblockrequest(hgc, "setumask2"); |
28160
098cb7bd46a7
chg: forward umask from client to server
Jun Wu <quark@fb.com>
parents:
28060
diff
changeset
|
453 } |
098cb7bd46a7
chg: forward umask from client to server
Jun Wu <quark@fb.com>
parents:
28060
diff
changeset
|
454 |
28060 | 455 /*! |
456 * Open connection to per-user cmdserver | |
457 * | |
458 * If no background server running, returns NULL. | |
459 */ | |
460 hgclient_t *hgc_open(const char *sockname) | |
461 { | |
462 int fd = socket(AF_UNIX, SOCK_STREAM, 0); | |
41336
763b45bc4483
cleanup: use clang-tidy to add missing {} around one-line statements
Augie Fackler <augie@google.com>
parents:
40109
diff
changeset
|
463 if (fd < 0) { |
28789
7f6e0a15189b
chg: replace abortmsg showing errno with abortmsgerrno
Jun Wu <quark@fb.com>
parents:
28769
diff
changeset
|
464 abortmsgerrno("cannot create socket"); |
41336
763b45bc4483
cleanup: use clang-tidy to add missing {} around one-line statements
Augie Fackler <augie@google.com>
parents:
40109
diff
changeset
|
465 } |
28060 | 466 |
467 /* don't keep fd on fork(), so that it can be closed when the parent | |
468 * process get terminated. */ | |
28855
f5764e177bbe
chg: extract the logic of setting FD_CLOEXEC to a utility function
Jun Wu <quark@fb.com>
parents:
28789
diff
changeset
|
469 fsetcloexec(fd); |
28060 | 470 |
471 struct sockaddr_un addr; | |
472 addr.sun_family = AF_UNIX; | |
30675
112915e9a363
chg: let hgc_open support long path
Jun Wu <quark@fb.com>
parents:
29719
diff
changeset
|
473 |
112915e9a363
chg: let hgc_open support long path
Jun Wu <quark@fb.com>
parents:
29719
diff
changeset
|
474 /* use chdir to workaround small sizeof(sun_path) */ |
112915e9a363
chg: let hgc_open support long path
Jun Wu <quark@fb.com>
parents:
29719
diff
changeset
|
475 int bakfd = -1; |
112915e9a363
chg: let hgc_open support long path
Jun Wu <quark@fb.com>
parents:
29719
diff
changeset
|
476 const char *basename = sockname; |
112915e9a363
chg: let hgc_open support long path
Jun Wu <quark@fb.com>
parents:
29719
diff
changeset
|
477 { |
112915e9a363
chg: let hgc_open support long path
Jun Wu <quark@fb.com>
parents:
29719
diff
changeset
|
478 const char *split = strrchr(sockname, '/'); |
112915e9a363
chg: let hgc_open support long path
Jun Wu <quark@fb.com>
parents:
29719
diff
changeset
|
479 if (split && split != sockname) { |
41336
763b45bc4483
cleanup: use clang-tidy to add missing {} around one-line statements
Augie Fackler <augie@google.com>
parents:
40109
diff
changeset
|
480 if (split[1] == '\0') { |
30675
112915e9a363
chg: let hgc_open support long path
Jun Wu <quark@fb.com>
parents:
29719
diff
changeset
|
481 abortmsg("sockname cannot end with a slash"); |
41336
763b45bc4483
cleanup: use clang-tidy to add missing {} around one-line statements
Augie Fackler <augie@google.com>
parents:
40109
diff
changeset
|
482 } |
30675
112915e9a363
chg: let hgc_open support long path
Jun Wu <quark@fb.com>
parents:
29719
diff
changeset
|
483 size_t len = split - sockname; |
112915e9a363
chg: let hgc_open support long path
Jun Wu <quark@fb.com>
parents:
29719
diff
changeset
|
484 char sockdir[len + 1]; |
112915e9a363
chg: let hgc_open support long path
Jun Wu <quark@fb.com>
parents:
29719
diff
changeset
|
485 memcpy(sockdir, sockname, len); |
112915e9a363
chg: let hgc_open support long path
Jun Wu <quark@fb.com>
parents:
29719
diff
changeset
|
486 sockdir[len] = '\0'; |
112915e9a363
chg: let hgc_open support long path
Jun Wu <quark@fb.com>
parents:
29719
diff
changeset
|
487 |
112915e9a363
chg: let hgc_open support long path
Jun Wu <quark@fb.com>
parents:
29719
diff
changeset
|
488 bakfd = open(".", O_DIRECTORY); |
41336
763b45bc4483
cleanup: use clang-tidy to add missing {} around one-line statements
Augie Fackler <augie@google.com>
parents:
40109
diff
changeset
|
489 if (bakfd == -1) { |
30675
112915e9a363
chg: let hgc_open support long path
Jun Wu <quark@fb.com>
parents:
29719
diff
changeset
|
490 abortmsgerrno("cannot open cwd"); |
41336
763b45bc4483
cleanup: use clang-tidy to add missing {} around one-line statements
Augie Fackler <augie@google.com>
parents:
40109
diff
changeset
|
491 } |
30675
112915e9a363
chg: let hgc_open support long path
Jun Wu <quark@fb.com>
parents:
29719
diff
changeset
|
492 |
112915e9a363
chg: let hgc_open support long path
Jun Wu <quark@fb.com>
parents:
29719
diff
changeset
|
493 int r = chdir(sockdir); |
41336
763b45bc4483
cleanup: use clang-tidy to add missing {} around one-line statements
Augie Fackler <augie@google.com>
parents:
40109
diff
changeset
|
494 if (r != 0) { |
30675
112915e9a363
chg: let hgc_open support long path
Jun Wu <quark@fb.com>
parents:
29719
diff
changeset
|
495 abortmsgerrno("cannot chdir %s", sockdir); |
41336
763b45bc4483
cleanup: use clang-tidy to add missing {} around one-line statements
Augie Fackler <augie@google.com>
parents:
40109
diff
changeset
|
496 } |
30675
112915e9a363
chg: let hgc_open support long path
Jun Wu <quark@fb.com>
parents:
29719
diff
changeset
|
497 |
112915e9a363
chg: let hgc_open support long path
Jun Wu <quark@fb.com>
parents:
29719
diff
changeset
|
498 basename = split + 1; |
112915e9a363
chg: let hgc_open support long path
Jun Wu <quark@fb.com>
parents:
29719
diff
changeset
|
499 } |
112915e9a363
chg: let hgc_open support long path
Jun Wu <quark@fb.com>
parents:
29719
diff
changeset
|
500 } |
41336
763b45bc4483
cleanup: use clang-tidy to add missing {} around one-line statements
Augie Fackler <augie@google.com>
parents:
40109
diff
changeset
|
501 if (strlen(basename) >= sizeof(addr.sun_path)) { |
30675
112915e9a363
chg: let hgc_open support long path
Jun Wu <quark@fb.com>
parents:
29719
diff
changeset
|
502 abortmsg("sockname is too long: %s", basename); |
41336
763b45bc4483
cleanup: use clang-tidy to add missing {} around one-line statements
Augie Fackler <augie@google.com>
parents:
40109
diff
changeset
|
503 } |
30675
112915e9a363
chg: let hgc_open support long path
Jun Wu <quark@fb.com>
parents:
29719
diff
changeset
|
504 strncpy(addr.sun_path, basename, sizeof(addr.sun_path)); |
28060 | 505 addr.sun_path[sizeof(addr.sun_path) - 1] = '\0'; |
506 | |
30675
112915e9a363
chg: let hgc_open support long path
Jun Wu <quark@fb.com>
parents:
29719
diff
changeset
|
507 /* real connect */ |
28060 | 508 int r = connect(fd, (struct sockaddr *)&addr, sizeof(addr)); |
30679
fe11f466880d
chg: handle connect failure before errno gets overridden
Jun Wu <quark@fb.com>
parents:
30675
diff
changeset
|
509 if (r < 0) { |
41336
763b45bc4483
cleanup: use clang-tidy to add missing {} around one-line statements
Augie Fackler <augie@google.com>
parents:
40109
diff
changeset
|
510 if (errno != ENOENT && errno != ECONNREFUSED) { |
30679
fe11f466880d
chg: handle connect failure before errno gets overridden
Jun Wu <quark@fb.com>
parents:
30675
diff
changeset
|
511 abortmsgerrno("cannot connect to %s", sockname); |
41336
763b45bc4483
cleanup: use clang-tidy to add missing {} around one-line statements
Augie Fackler <augie@google.com>
parents:
40109
diff
changeset
|
512 } |
30679
fe11f466880d
chg: handle connect failure before errno gets overridden
Jun Wu <quark@fb.com>
parents:
30675
diff
changeset
|
513 } |
30675
112915e9a363
chg: let hgc_open support long path
Jun Wu <quark@fb.com>
parents:
29719
diff
changeset
|
514 if (bakfd != -1) { |
112915e9a363
chg: let hgc_open support long path
Jun Wu <quark@fb.com>
parents:
29719
diff
changeset
|
515 fchdirx(bakfd); |
112915e9a363
chg: let hgc_open support long path
Jun Wu <quark@fb.com>
parents:
29719
diff
changeset
|
516 close(bakfd); |
112915e9a363
chg: let hgc_open support long path
Jun Wu <quark@fb.com>
parents:
29719
diff
changeset
|
517 } |
28060 | 518 if (r < 0) { |
519 close(fd); | |
30679
fe11f466880d
chg: handle connect failure before errno gets overridden
Jun Wu <quark@fb.com>
parents:
30675
diff
changeset
|
520 return NULL; |
28060 | 521 } |
28769
222f482930c8
chg: make connect debug message less repetitive
Jun Wu <quark@fb.com>
parents:
28551
diff
changeset
|
522 debugmsg("connected to %s", addr.sun_path); |
28060 | 523 |
28166
c6e0c2533e5f
chg: use mallocx and reallocx in hgclient
Jun Wu <quark@fb.com>
parents:
28160
diff
changeset
|
524 hgclient_t *hgc = mallocx(sizeof(hgclient_t)); |
28060 | 525 memset(hgc, 0, sizeof(*hgc)); |
526 hgc->sockfd = fd; | |
527 initcontext(&hgc->ctx); | |
528 | |
529 readhello(hgc); | |
41336
763b45bc4483
cleanup: use clang-tidy to add missing {} around one-line statements
Augie Fackler <augie@google.com>
parents:
40109
diff
changeset
|
530 if (!(hgc->capflags & CAP_RUNCOMMAND)) { |
28060 | 531 abortmsg("insufficient capability: runcommand"); |
41336
763b45bc4483
cleanup: use clang-tidy to add missing {} around one-line statements
Augie Fackler <augie@google.com>
parents:
40109
diff
changeset
|
532 } |
763b45bc4483
cleanup: use clang-tidy to add missing {} around one-line statements
Augie Fackler <augie@google.com>
parents:
40109
diff
changeset
|
533 if (hgc->capflags & CAP_SETPROCNAME) { |
30751 | 534 updateprocname(hgc); |
41336
763b45bc4483
cleanup: use clang-tidy to add missing {} around one-line statements
Augie Fackler <augie@google.com>
parents:
40109
diff
changeset
|
535 } |
763b45bc4483
cleanup: use clang-tidy to add missing {} around one-line statements
Augie Fackler <augie@google.com>
parents:
40109
diff
changeset
|
536 if (hgc->capflags & CAP_ATTACHIO) { |
28060 | 537 attachio(hgc); |
41336
763b45bc4483
cleanup: use clang-tidy to add missing {} around one-line statements
Augie Fackler <augie@google.com>
parents:
40109
diff
changeset
|
538 } |
763b45bc4483
cleanup: use clang-tidy to add missing {} around one-line statements
Augie Fackler <augie@google.com>
parents:
40109
diff
changeset
|
539 if (hgc->capflags & CAP_CHDIR) { |
28060 | 540 chdirtocwd(hgc); |
41336
763b45bc4483
cleanup: use clang-tidy to add missing {} around one-line statements
Augie Fackler <augie@google.com>
parents:
40109
diff
changeset
|
541 } |
763b45bc4483
cleanup: use clang-tidy to add missing {} around one-line statements
Augie Fackler <augie@google.com>
parents:
40109
diff
changeset
|
542 if (hgc->capflags & CAP_SETUMASK2) { |
28160
098cb7bd46a7
chg: forward umask from client to server
Jun Wu <quark@fb.com>
parents:
28060
diff
changeset
|
543 forwardumask(hgc); |
41336
763b45bc4483
cleanup: use clang-tidy to add missing {} around one-line statements
Augie Fackler <augie@google.com>
parents:
40109
diff
changeset
|
544 } |
28060 | 545 |
546 return hgc; | |
547 } | |
548 | |
549 /*! | |
550 * Close connection and free allocated memory | |
551 */ | |
552 void hgc_close(hgclient_t *hgc) | |
553 { | |
554 assert(hgc); | |
555 freecontext(&hgc->ctx); | |
556 close(hgc->sockfd); | |
557 free(hgc); | |
558 } | |
559 | |
29581 | 560 pid_t hgc_peerpgid(const hgclient_t *hgc) |
561 { | |
562 assert(hgc); | |
563 return hgc->pgid; | |
564 } | |
565 | |
28060 | 566 pid_t hgc_peerpid(const hgclient_t *hgc) |
567 { | |
568 assert(hgc); | |
569 return hgc->pid; | |
570 } | |
571 | |
572 /*! | |
28356
a5c773acb018
chg: implement validate in hgclient
Jun Wu <quark@fb.com>
parents:
28166
diff
changeset
|
573 * Send command line arguments to let the server load the repo config and check |
a5c773acb018
chg: implement validate in hgclient
Jun Wu <quark@fb.com>
parents:
28166
diff
changeset
|
574 * whether it can process our request directly or not. |
a5c773acb018
chg: implement validate in hgclient
Jun Wu <quark@fb.com>
parents:
28166
diff
changeset
|
575 * Make sure hgc_setenv is called before calling this. |
a5c773acb018
chg: implement validate in hgclient
Jun Wu <quark@fb.com>
parents:
28166
diff
changeset
|
576 * |
a5c773acb018
chg: implement validate in hgclient
Jun Wu <quark@fb.com>
parents:
28166
diff
changeset
|
577 * @return - NULL, the server believes it can handle our request, or does not |
a5c773acb018
chg: implement validate in hgclient
Jun Wu <quark@fb.com>
parents:
28166
diff
changeset
|
578 * support "validate" command. |
28535
aa082a8125da
chgserver: add an explicit "reconnect" instruction to validate
Jun Wu <quark@fb.com>
parents:
28512
diff
changeset
|
579 * - a list of strings, the server probably cannot handle our request |
aa082a8125da
chgserver: add an explicit "reconnect" instruction to validate
Jun Wu <quark@fb.com>
parents:
28512
diff
changeset
|
580 * and it sent instructions telling us what to do next. See |
28356
a5c773acb018
chg: implement validate in hgclient
Jun Wu <quark@fb.com>
parents:
28166
diff
changeset
|
581 * chgserver.py for possible instruction formats. |
a5c773acb018
chg: implement validate in hgclient
Jun Wu <quark@fb.com>
parents:
28166
diff
changeset
|
582 * the list should be freed by the caller. |
a5c773acb018
chg: implement validate in hgclient
Jun Wu <quark@fb.com>
parents:
28166
diff
changeset
|
583 * the last string is guaranteed to be NULL. |
a5c773acb018
chg: implement validate in hgclient
Jun Wu <quark@fb.com>
parents:
28166
diff
changeset
|
584 */ |
a5c773acb018
chg: implement validate in hgclient
Jun Wu <quark@fb.com>
parents:
28166
diff
changeset
|
585 const char **hgc_validate(hgclient_t *hgc, const char *const args[], |
35959
9724f54923ec
chg: enable clang-format on all .c and .h files
Augie Fackler <augie@google.com>
parents:
31941
diff
changeset
|
586 size_t argsize) |
28356
a5c773acb018
chg: implement validate in hgclient
Jun Wu <quark@fb.com>
parents:
28166
diff
changeset
|
587 { |
a5c773acb018
chg: implement validate in hgclient
Jun Wu <quark@fb.com>
parents:
28166
diff
changeset
|
588 assert(hgc); |
41336
763b45bc4483
cleanup: use clang-tidy to add missing {} around one-line statements
Augie Fackler <augie@google.com>
parents:
40109
diff
changeset
|
589 if (!(hgc->capflags & CAP_VALIDATE)) { |
28356
a5c773acb018
chg: implement validate in hgclient
Jun Wu <quark@fb.com>
parents:
28166
diff
changeset
|
590 return NULL; |
41336
763b45bc4483
cleanup: use clang-tidy to add missing {} around one-line statements
Augie Fackler <augie@google.com>
parents:
40109
diff
changeset
|
591 } |
28356
a5c773acb018
chg: implement validate in hgclient
Jun Wu <quark@fb.com>
parents:
28166
diff
changeset
|
592 |
a5c773acb018
chg: implement validate in hgclient
Jun Wu <quark@fb.com>
parents:
28166
diff
changeset
|
593 packcmdargs(&hgc->ctx, args, argsize); |
a5c773acb018
chg: implement validate in hgclient
Jun Wu <quark@fb.com>
parents:
28166
diff
changeset
|
594 writeblockrequest(hgc, "validate"); |
a5c773acb018
chg: implement validate in hgclient
Jun Wu <quark@fb.com>
parents:
28166
diff
changeset
|
595 handleresponse(hgc); |
a5c773acb018
chg: implement validate in hgclient
Jun Wu <quark@fb.com>
parents:
28166
diff
changeset
|
596 |
a5c773acb018
chg: implement validate in hgclient
Jun Wu <quark@fb.com>
parents:
28166
diff
changeset
|
597 /* the server returns '\0' if it can handle our request */ |
41336
763b45bc4483
cleanup: use clang-tidy to add missing {} around one-line statements
Augie Fackler <augie@google.com>
parents:
40109
diff
changeset
|
598 if (hgc->ctx.datasize <= 1) { |
28356
a5c773acb018
chg: implement validate in hgclient
Jun Wu <quark@fb.com>
parents:
28166
diff
changeset
|
599 return NULL; |
41336
763b45bc4483
cleanup: use clang-tidy to add missing {} around one-line statements
Augie Fackler <augie@google.com>
parents:
40109
diff
changeset
|
600 } |
28356
a5c773acb018
chg: implement validate in hgclient
Jun Wu <quark@fb.com>
parents:
28166
diff
changeset
|
601 |
a5c773acb018
chg: implement validate in hgclient
Jun Wu <quark@fb.com>
parents:
28166
diff
changeset
|
602 /* make sure the buffer is '\0' terminated */ |
a5c773acb018
chg: implement validate in hgclient
Jun Wu <quark@fb.com>
parents:
28166
diff
changeset
|
603 enlargecontext(&hgc->ctx, hgc->ctx.datasize + 1); |
a5c773acb018
chg: implement validate in hgclient
Jun Wu <quark@fb.com>
parents:
28166
diff
changeset
|
604 hgc->ctx.data[hgc->ctx.datasize] = '\0'; |
a5c773acb018
chg: implement validate in hgclient
Jun Wu <quark@fb.com>
parents:
28166
diff
changeset
|
605 return unpackcmdargsnul(&hgc->ctx); |
a5c773acb018
chg: implement validate in hgclient
Jun Wu <quark@fb.com>
parents:
28166
diff
changeset
|
606 } |
a5c773acb018
chg: implement validate in hgclient
Jun Wu <quark@fb.com>
parents:
28166
diff
changeset
|
607 |
a5c773acb018
chg: implement validate in hgclient
Jun Wu <quark@fb.com>
parents:
28166
diff
changeset
|
608 /*! |
28060 | 609 * Execute the specified Mercurial command |
610 * | |
611 * @return result code | |
612 */ | |
613 int hgc_runcommand(hgclient_t *hgc, const char *const args[], size_t argsize) | |
614 { | |
615 assert(hgc); | |
616 | |
617 packcmdargs(&hgc->ctx, args, argsize); | |
618 writeblockrequest(hgc, "runcommand"); | |
619 handleresponse(hgc); | |
620 | |
621 int32_t exitcode_n; | |
622 if (hgc->ctx.datasize != sizeof(exitcode_n)) { | |
623 abortmsg("unexpected size of exitcode"); | |
624 } | |
625 memcpy(&exitcode_n, hgc->ctx.data, sizeof(exitcode_n)); | |
626 return ntohl(exitcode_n); | |
627 } | |
628 | |
629 /*! | |
630 * (Re-)send client's stdio channels so that the server can access to tty | |
631 */ | |
632 void hgc_attachio(hgclient_t *hgc) | |
633 { | |
634 assert(hgc); | |
41336
763b45bc4483
cleanup: use clang-tidy to add missing {} around one-line statements
Augie Fackler <augie@google.com>
parents:
40109
diff
changeset
|
635 if (!(hgc->capflags & CAP_ATTACHIO)) { |
28060 | 636 return; |
41336
763b45bc4483
cleanup: use clang-tidy to add missing {} around one-line statements
Augie Fackler <augie@google.com>
parents:
40109
diff
changeset
|
637 } |
28060 | 638 attachio(hgc); |
639 } | |
640 | |
641 /*! | |
642 * Update server's environment variables | |
643 * | |
644 * @param envp list of environment variables in "NAME=VALUE" format, | |
645 * terminated by NULL. | |
646 */ | |
647 void hgc_setenv(hgclient_t *hgc, const char *const envp[]) | |
648 { | |
649 assert(hgc && envp); | |
41336
763b45bc4483
cleanup: use clang-tidy to add missing {} around one-line statements
Augie Fackler <augie@google.com>
parents:
40109
diff
changeset
|
650 if (!(hgc->capflags & CAP_SETENV)) { |
28060 | 651 return; |
41336
763b45bc4483
cleanup: use clang-tidy to add missing {} around one-line statements
Augie Fackler <augie@google.com>
parents:
40109
diff
changeset
|
652 } |
28060 | 653 packcmdargs(&hgc->ctx, envp, /*argsize*/ -1); |
654 writeblockrequest(hgc, "setenv"); | |
655 } |