30 Commits
Author SHA1 Message Date
Konrad Dybcio f803b4d573 meson.build: Set v1.1 version
Signed-off-by: Konrad Dybcio <konrad.dybcio@linaro.org>
2024-07-23 14:22:35 +02:00
Stefan Hansson 0197df8e61 Add support for zstd-compressed readonly files
We need this to load compressed files from linux-firmware.

Closes https://github.com/linux-msm/tqftpserv/issues/19

Signed-off-by: Stefan Hansson <newbyte@postmarketos.org>
Tested-by: Nikita Travkin <nikita@trvn.ru>
2024-07-23 14:20:00 +02:00
Alexey Minnekhanov 47e8957271 meson.build: Make systemd dep optional
Some distros still don't use SystemD (yet), make it optional.

It is not required to build the binary, just to install systemd
unit file.

Signed-off-by: Alexey Minnekhanov <alexey.min@gmail.com>
2024-04-30 23:51:30 +02:00
Konrad Dybcio 5f9f263ae2 treewide: meson 2024-04-23 19:38:37 +02:00
Konrad Dybcio 3a10a7dd6f treewide: use SPDX license IDs 2024-04-08 23:53:19 +02:00
Luca Weiss a09948e5e3 tqftpserv: don't print "End of Transfer" as an error
On newer modems, the firmware seems to use a stat-like operation
relatively often which end the transfer with error 9 "End of Transfer".

In practise we're getting a RRQ with the tsize option set, we'll query
the file size from the filesystem, respond with the updated tsize in
OACK, then the modem will send us OP_ERROR with code 9 "End of
Transfer".

Since that's expected in this operation, let's not make it look like an
error to not confuse users/developers trying to debug the modem.
2024-04-08 23:17:36 +02:00
Luca Weiss 588a8d3c8f tqftpserv: add rsize & offset options to RRQ print
Since new modems send multiple file requests with different rsize and
offset on a single file, it's useful to print those also.
2024-04-08 23:17:36 +02:00
Luca Weiss e167336fca tqftpserv: handle rsize & offset options
The rsize (number of bytes the client wants to receive) and offset (the
offset in bytes from the start of file) are used on newer modems (such
as SM7225 or QCM6490), so let's support them properly.

Some examples to illustrate the usage:
* rsize=160 seek=52: we should send 160 bytes starting at byte 52 in the
  requested file. With blksize=7680 this is a single 160(+4) sized
  packet.
* rsize=313028 seek=81920: we should send in total 313028 bytes,
  starting at byte 81920. With blksize=7680 and wsize=10 this spans
  multiple windows of 10 packets each, that example would be packed into
  41 packets with an ACK between every 10 windows - and a final ACK at
  the end.
* rsize=53760 blksize=7680: We send 7 packets of 7680 bytes each, then
  we send 1 packet with 0 bytes of data (only 4 bytes header) and then
  get an ACK on packet 8.
2024-04-08 23:17:36 +02:00
Luca Weiss 69c7cfcf73 tqftpserv: add seek option parsing
Actual handling will be added in follow-up commits as rsize is also not
currently handled, and both are quite interconnected.
2024-04-08 23:17:36 +02:00
Luca Weiss 2dd48a2899 tqftpserv: allow sending data packet with 0-byte data payload
If the client requests a file that's completely empty, it makes sense to
send a response to that request with - well - 0 bytes of data and just
the 4-byte header.

But also if the client requests for example a file of rsize=53760 and
blksize=7680, then will send 7 full packets of data in the window, but
afterwards we still need to send an empty packet (just the 4 bytes of
header) to make sure the client understands that we've sent all the
requested data. Otherwise it's going to time out and re-request the
blocks and we're stuck in a loop.

So consider pread return value of 0 to not be an error and send a
response packet back.
2024-04-08 23:17:36 +02:00
Luca Weiss 98d162c9ff translate: Fix firmware path for 'fallback' case
The dirname() function can and will modify the parameter, so if we call
dirname() twice on the same firmware_value parameter we will get
separate results messing up our names.

Copy the firmware_value value before passing it to dirname(). Also let's
just do this operation once to save copy-pasting some code.

Fixes: a4c755d ("Use firmware path from sysfs")
2024-04-08 22:31:59 +02:00
Konrad Dybcio 37669ab1e2 Merge pull request #12 from minlexx/respect-firmware-sysfs-path
Use firmware path from sysfs
2024-03-27 21:27:08 +01:00
Alexey Minnekhanov a4c755db17 Use firmware path from sysfs
Some linux distributions adjust path fro kernel to load
firmware from using /sys/module/firmware_class/parameters/path.
Kernel supports it, teach tqftpserv to respect it too.

Signed-off-by: Alexey Minnekhanov <alexeymin@postmarketos.org>
2024-03-24 00:00:31 +03:00
Bjorn Andersson 289214a5d1 Merge pull request #11 from z3ntu/misc-improvements
Some improvements for error handling
2024-01-22 09:57:48 -06:00
Luca Weiss 00fe2775de tqftpserv: add docs for options
The explanation is spread throughout various RFCs or the code - or at
least for rsize based on strace'ing the communication.
2024-01-19 11:52:32 +01:00
Luca Weiss fb20008c86 translate: don't try to close already closed class_fd
After calling fdopendir "the file descriptor is under the control of the
system" and calling closedir(class_dir) already closes class_fd, so
let's not try to close it again.

Just close class_fd when fdopendir fails and remove the close call from
the regular exit path.
2024-01-18 15:33:46 +01:00
Luca Weiss a540a1c307 tqftpserv: also print value for unknown options 2023-02-10 13:17:28 +01:00
Luca Weiss 3fa49eafe0 tqftpserv: handle OP_ERROR code
When we get an error from the remote we should print it, for example:

  [TQFTP] received error:  8 - Expected OACK for UNLINK
2023-02-10 12:13:00 +01:00
Luca Weiss aaba76c6ba tqftpserv: also print error code on read error
Along with the error message we also get a uint16 error code that we can
print.
2023-02-10 12:10:46 +01:00
John Stultz de42697a24 translate: Null terminate firmware_value string
Null terminate firmware_value string to prevent
strlen() read past the end of buffer.

Change-Id: I57cd01c4d0c58c6057872bc4b36129f09bc034a3
Signed-off-by: Amit Pundir <amit.pundir@linaro.org>
Signed-off-by: John Stultz <john.stultz@linaro.org>
2023-01-17 22:57:14 -06:00
Amit Pundir 538efb3096 translate: Use /data/vendor for temporary files
On AOSP, /tmp is not available. Use /data/vendor instead
for temporary files.

Change-Id: I1d6ebd669573b3c3134e6da943ea9355ed83a3fe
Signed-off-by: Amit Pundir <amit.pundir@linaro.org>
Signed-off-by: John Stultz <john.stultz@linaro.org>
2023-01-17 22:57:14 -06:00
Sireesh Kodali 48b9e1cbb1 tqftpserv: add support for TFTP options on WRQs 2023-01-17 22:56:21 -06:00
Jami Kettunen 3488007880 Makefile: allow $(CFLAGS), $(LDFLAGS) override
The caller might have specified CFLAGS or LDFLAGS. Let's respect those.
2023-01-17 22:54:25 -06:00
Amit Pundir 783425b550 ANDROID: Add Android.bp makefile
Add Android.bp makefile to build tqftpserv for AOSP.

Signed-off-by: Amit Pundir <amit.pundir@linaro.org>
Signed-off-by: Bjorn Andersson <bjorn.andersson@linaro.org>
2020-02-07 10:43:39 -08:00
Khem Raj 2324458f18 include limits.h for PATH_MAX
Signed-off-by: Khem Raj <raj.khem@gmail.com>
2019-12-02 13:45:16 -08:00
Bjorn Andersson fe53d2a810 tqftpserv: Implement path translation
The tftp clients in the modem requests files from either /readwrite or
/readonly/firmware/image, but as the wlanmdsp.mbn file requested is
signed with a vendor specific key these paths needs to be translated.

/readwrite is translated to /tmp/tqftpserv for now and the
/readonly/firmware/image is translated to a search for the requested
file alongside any of the remoteproc firmware files. The result is that
the wlanmdsp.mbn can be stored in linux-firmware alongside it's
device/vendor specific modem firmware.

Signed-off-by: Bjorn Andersson <bjorn.andersson@linaro.org>
2019-10-26 11:14:03 -07:00
Bjorn Andersson 75a7a89398 tqftpserv.service: Add systemd service
Add tqftpserv.service and install this.

Signed-off-by: Bjorn Andersson <bjorn.andersson@linaro.org>
2019-07-25 11:36:16 -07:00
Bjorn Andersson 6348c52f20 tqftpserv: Remove the spamming of stdout
Some mechanism for toggling the verbosity of the prints should be
introduced, but for now just comment out all the debug prints.

Signed-off-by: Bjorn Andersson <bjorn.andersson@linaro.org>
2019-06-10 16:40:11 -07:00
Bjorn Andersson 7167c88ead tqftpserv: Don't close socket at EOF
When during a read we close the socket as we send the last block the
remote will be notified about the closure and consider the transfer
incomplete. Leave the socket open until we receive a EOF error from the
remote side.

Signed-off-by: Bjorn Andersson <bjorn.andersson@linaro.org>
2019-06-10 16:36:13 -07:00
Bjorn Andersson 2ae2758cba Add LICENSE file
Signed-off-by: Bjorn Andersson <bjorn.andersson@linaro.org>
2019-03-15 11:34:42 -07:00
12 changed files with 618 additions and 137 deletions

No files matched your search

+9
View File
@@ -0,0 +1,9 @@
cc_binary {
name: "tqftpserv",
vendor: true,
srcs: [
"tqftpserv.c",
"translate.c",
],
shared_libs: ["libqrtr"],
}
+30
View File
@@ -0,0 +1,30 @@
/*
* Copyright (c) 2018, Linaro Ltd.
* All rights reserved.
*
* Redistribution and use in source and binary forms, with or without
* modification, are permitted provided that the following conditions are met:
*
* 1. Redistributions of source code must retain the above copyright notice,
* this list of conditions and the following disclaimer.
*
* 2. Redistributions in binary form must reproduce the above copyright notice,
* this list of conditions and the following disclaimer in the documentation
* and/or other materials provided with the distribution.
*
* 3. Neither the name of the copyright holder nor the names of its contributors
* may be used to endorse or promote products derived from this software without
* specific prior written permission.
*
* THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS "AS IS"
* AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE
* IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE
* ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT HOLDER OR CONTRIBUTORS BE
* LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR
* CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF
* SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS
* INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN
* CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE)
* ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE
* POSSIBILITY OF SUCH DAMAGE.
*/
-17
View File
@@ -1,17 +0,0 @@
TQFTPSERV := tqftpserv
CFLAGS := -Wall -g -O2
LDFLAGS := -lqrtr
SRCS := tqftpserv.c
OBJS := $(SRCS:.c=.o)
$(TQFTPSERV): $(OBJS)
$(CC) -o $@ $^ $(LDFLAGS)
install: $(TQFTPSERV)
install -D -m 755 $< $(DESTDIR)$(prefix)/bin/$<
clean:
rm -f $(TQFTPSERV) $(OBJS)
+1 -27
View File
@@ -1,32 +1,6 @@
// SPDX-License-Identifier: BSD-3-Clause
/*
* Copyright (c) 2016, Linaro Ltd.
* All rights reserved.
*
* Redistribution and use in source and binary forms, with or without
* modification, are permitted provided that the following conditions are met:
*
* 1. Redistributions of source code must retain the above copyright notice,
* this list of conditions and the following disclaimer.
*
* 2. Redistributions in binary form must reproduce the above copyright notice,
* this list of conditions and the following disclaimer in the documentation
* and/or other materials provided with the distribution.
*
* 3. Neither the name of the copyright holder nor the names of its contributors
* may be used to endorse or promote products derived from this software without
* specific prior written permission.
*
* THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS "AS IS"
* AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE
* IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE
* ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT HOLDER OR CONTRIBUTORS BE
* LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR
* CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF
* SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS
* INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN
* CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE)
* ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE
* POSSIBILITY OF SUCH DAMAGE.
*/
#ifndef __LIST_H__
#define __LIST_H__
+44
View File
@@ -0,0 +1,44 @@
# SPDX-License-Identifier: BSD-3-Clause
project('tqftpserv',
'c',
version: '1.1',
default_options: [
'warning_level=1',
'buildtype=release',
])
prefix = get_option('prefix')
zstd_dep = dependency('libzstd')
# Not required to build the executable, only to install unit file
systemd = dependency('systemd', required : false)
if systemd.found()
systemd_system_unit_dir = get_option('systemd-unit-prefix')
if systemd_system_unit_dir == ''
systemd_system_unit_dir = systemd.get_variable(
pkgconfig : 'systemdsystemunitdir',
pkgconfig_define: ['prefix', prefix])
endif
endif
qrtr_dep = dependency('qrtr')
tqftpserv_srcs = ['translate.c',
'tqftpserv.c',
'zstd-decompress.c']
executable('tqftpserv',
tqftpserv_srcs,
dependencies : [qrtr_dep, zstd_dep],
install : true)
if systemd.found()
systemd_unit_conf = configuration_data()
systemd_unit_conf.set('prefix', prefix)
configure_file(
input : 'tqftpserv.service.in',
output : 'tqftpserv.service',
configuration : systemd_unit_conf,
install_dir : systemd_system_unit_dir)
endif
+4
View File
@@ -0,0 +1,4 @@
option('systemd-unit-prefix',
type: 'string',
description: 'Directory for systemd system unit files'
)
+161 -93
View File
@@ -1,32 +1,6 @@
// SPDX-License-Identifier: BSD-3-Clause
/*
* Copyright (c) 2018, Linaro Ltd.
* All rights reserved.
*
* Redistribution and use in source and binary forms, with or without
* modification, are permitted provided that the following conditions are met:
*
* 1. Redistributions of source code must retain the above copyright notice,
* this list of conditions and the following disclaimer.
*
* 2. Redistributions in binary form must reproduce the above copyright notice,
* this list of conditions and the following disclaimer in the documentation
* and/or other materials provided with the distribution.
*
* 3. Neither the name of the copyright holder nor the names of its contributors
* may be used to endorse or promote products derived from this software without
* specific prior written permission.
*
* THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS "AS IS"
* AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE
* IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE
* ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT HOLDER OR CONTRIBUTORS BE
* LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR
* CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF
* SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS
* INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN
* CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE)
* ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE
* POSSIBILITY OF SUCH DAMAGE.
*/
#include <arpa/inet.h>
#include <sys/stat.h>
@@ -40,8 +14,8 @@
#include <unistd.h>
#include "list.h"
#define BASE_DIR "/"
#include "translate.h"
#include "zstd-decompress.h"
#define MAX(x, y) ((x) > (y) ? (x) : (y))
@@ -54,6 +28,10 @@ enum {
OP_OACK,
};
enum {
ERROR_END_OF_TRANSFER = 9,
};
struct tftp_client {
struct list_head node;
@@ -68,15 +46,17 @@ struct tftp_client {
size_t rsize;
size_t wsize;
unsigned int timeoutms;
off_t seek;
};
static struct list_head readers = LIST_INIT(readers);
static struct list_head writers = LIST_INIT(writers);
static ssize_t tftp_send_data(struct tftp_client *client,
unsigned int block, size_t offset)
unsigned int block, size_t offset, size_t response_size)
{
ssize_t len;
size_t send_len;
char *buf;
char *p;
@@ -90,17 +70,29 @@ static ssize_t tftp_send_data(struct tftp_client *client,
*p++ = block & 0xff;
len = pread(client->fd, p, client->blksize, offset);
if (len <= 0) {
if (len < 0)
printf("[TQFTP] failed to read data\n");
if (len < 0) {
printf("[TQFTP] failed to read data\n");
free(buf);
return len;
}
p += len;
printf("[TQFTP] Sending %zd bytes of DATA\n", p - buf);
len = send(client->sock, buf, p - buf, 0);
/* If rsize was set, we should limit the data in the response to n bytes */
if (response_size != 0) {
/* Header (4 bytes) + data size */
send_len = 4 + response_size;
if (send_len > p - buf) {
printf("[TQFTP] requested data of %ld bytes but only read %ld bytes from file, rejecting\n", response_size, len);
free(buf);
return -EINVAL;
}
} else {
send_len = p - buf;
}
// printf("[TQFTP] Sending %zd bytes of DATA\n", send_len);
len = send(client->sock, buf, send_len, 0);
free(buf);
@@ -119,7 +111,8 @@ static int tftp_send_ack(int sock, int block)
}
static int tftp_send_oack(int sock, size_t *blocksize, size_t *tsize,
size_t *wsize, unsigned int *timeoutms, size_t *rsize)
size_t *wsize, unsigned int *timeoutms, size_t *rsize,
off_t *seek)
{
char buf[512];
char *p = buf;
@@ -173,6 +166,15 @@ static int tftp_send_oack(int sock, size_t *blocksize, size_t *tsize,
*p++ = '\0';
}
if (seek) {
strcpy(p, "seek");
p += 5;
n = sprintf(p, "%zd", *seek);
p += n;
*p++ = '\0';
}
return send(sock, buf, p - buf, 0);
}
@@ -197,13 +199,53 @@ static int tftp_send_error(int sock, int code, const char *msg)
return rc;
}
static void parse_options(const char *buf, size_t len, size_t *blksize,
ssize_t *tsize, size_t *wsize, unsigned int *timeoutms,
size_t *rsize, off_t *seek)
{
const char *opt, *value;
const char *p = buf;
while (p < buf + len) {
/* XXX: ensure we're not running off the end */
opt = p;
p += strlen(p) + 1;
/* XXX: ensure we're not running off the end */
value = p;
p += strlen(p) + 1;
/*
* blksize: block size - how many bytes to send at once
* timeoutms: timeout in milliseconds
* tsize: total size - request to get file size in bytes
* rsize: read size - how many bytes to send, not full file
* wsize: window size - how many blocks to send without ACK
* seek: offset from beginning of file in bytes to start reading
*/
if (!strcmp(opt, "blksize")) {
*blksize = atoi(value);
} else if (!strcmp(opt, "timeoutms")) {
*timeoutms = atoi(value);
} else if (!strcmp(opt, "tsize")) {
*tsize = atoi(value);
} else if (!strcmp(opt, "rsize")) {
*rsize = atoi(value);
} else if (!strcmp(opt, "wsize")) {
*wsize = atoi(value);
} else if (!strcmp(opt, "seek")) {
*seek = atoi(value);
} else {
printf("[TQFTP] Ignoring unknown option '%s' with value '%s'\n", opt, value);
}
}
}
static void handle_rrq(const char *buf, size_t len, struct sockaddr_qrtr *sq)
{
struct tftp_client *client;
const char *filename;
const char *value;
const char *mode;
const char *opt;
struct stat sb;
const char *p;
ssize_t tsize = -1;
@@ -211,8 +253,8 @@ static void handle_rrq(const char *buf, size_t len, struct sockaddr_qrtr *sq)
unsigned int timeoutms = 1000;
size_t rsize = 0;
size_t wsize = 0;
off_t seek = 0;
bool do_oack = false;
char path[256];
int sock;
int ret;
int fd;
@@ -231,36 +273,14 @@ static void handle_rrq(const char *buf, size_t len, struct sockaddr_qrtr *sq)
return;
}
printf("[TQFTP] RRQ: %s (%s)\n", filename, mode);
if (p < buf + len) {
do_oack = true;
while (p < buf + len) {
/* XXX: ensure we're not running off the end */
opt = p;
p += strlen(p) + 1;
/* XXX: ensure we're not running off the end */
value = p;
p += strlen(p) + 1;
if (!strcmp(opt, "blksize")) {
blksize = atoi(value);
} else if (!strcmp(opt, "timeoutms")) {
timeoutms = atoi(value);
} else if (!strcmp(opt, "tsize")) {
tsize = atoi(value);
} else if (!strcmp(opt, "rsize")) {
rsize = atoi(value);
} else if (!strcmp(opt, "wsize")) {
wsize = atoi(value);
} else {
printf("[TQFTP] Ignoring unknown option '%s'\n", opt);
}
}
parse_options(p, len - (p - buf), &blksize, &tsize, &wsize,
&timeoutms, &rsize, &seek);
}
printf("[TQFTP] RRQ: %s (mode=%s rsize=%ld seek=%ld)\n", filename, mode, rsize, seek);
sock = qrtr_open(0);
if (sock < 0) {
/* XXX: error */
@@ -275,10 +295,9 @@ static void handle_rrq(const char *buf, size_t len, struct sockaddr_qrtr *sq)
return;
}
snprintf(path, sizeof(path), BASE_DIR "/%s", filename);
fd = open(path, O_RDONLY);
fd = translate_open(filename, O_RDONLY);
if (fd < 0) {
printf("[TQFTP] unable to open %s (%d), reject\n", path, errno);
printf("[TQFTP] unable to open %s (%d), reject\n", filename, errno);
tftp_send_error(sock, 1, "file not found");
return;
}
@@ -296,8 +315,9 @@ static void handle_rrq(const char *buf, size_t len, struct sockaddr_qrtr *sq)
client->rsize = rsize;
client->wsize = wsize;
client->timeoutms = timeoutms;
client->seek = seek;
printf("[TQFTP] new reader added\n");
// printf("[TQFTP] new reader added\n");
list_add(&readers, &client->node);
@@ -306,9 +326,10 @@ static void handle_rrq(const char *buf, size_t len, struct sockaddr_qrtr *sq)
tsize ? (size_t*)&tsize : NULL,
wsize ? &wsize : NULL,
&client->timeoutms,
rsize ? &rsize: NULL);
rsize ? &rsize : NULL,
seek ? &seek : NULL);
} else {
tftp_send_data(client, 1, 0);
tftp_send_data(client, 1, 0, 0);
}
}
@@ -317,13 +338,21 @@ static void handle_wrq(const char *buf, size_t len, struct sockaddr_qrtr *sq)
struct tftp_client *client;
const char *filename;
const char *mode;
char path[256];
const char *p;
ssize_t tsize = -1;
size_t blksize = 512;
unsigned int timeoutms = 1000;
size_t rsize = 0;
size_t wsize = 0;
off_t seek = 0;
bool do_oack = false;
int sock;
int ret;
int fd;
filename = buf + 2;
mode = buf + 2 + strlen(filename) + 1;
p = mode + strlen(mode) + 1;
if (strcasecmp(mode, "octet")) {
/* XXX: error */
@@ -333,11 +362,16 @@ static void handle_wrq(const char *buf, size_t len, struct sockaddr_qrtr *sq)
printf("[TQFTP] WRQ: %s (%s)\n", filename, mode);
snprintf(path, sizeof(path), BASE_DIR "/%s", filename);
fd = open(path, O_WRONLY | O_CREAT, 0666);
if (p < buf + len) {
do_oack = true;
parse_options(p, len - (p - buf), &blksize, &tsize, &wsize,
&timeoutms, &rsize, &seek);
}
fd = translate_open(filename, O_WRONLY | O_CREAT);
if (fd < 0) {
/* XXX: error */
printf("[TQFTP] unable to open %s (%d), reject\n", path, errno);
printf("[TQFTP] unable to open %s (%d), reject\n", filename, errno);
return;
}
@@ -359,19 +393,26 @@ static void handle_wrq(const char *buf, size_t len, struct sockaddr_qrtr *sq)
client->sq = *sq;
client->sock = sock;
client->fd = fd;
client->blksize = blksize;
client->rsize = rsize;
client->wsize = wsize;
client->timeoutms = timeoutms;
client->seek = seek;
ret = tftp_send_ack(client->sock, 0);
if (ret < 0) {
printf("[TQFTP] unable to send ack\n");
close(sock);
close(fd);
free(client);
return;
}
printf("[TQFTP] new writer added\n");
// printf("[TQFTP] new writer added\n");
list_add(&writers, &client->node);
if (do_oack) {
tftp_send_oack(client->sock, &blksize,
tsize ? (size_t*)&tsize : NULL,
wsize ? &wsize : NULL,
&client->timeoutms,
rsize ? &rsize : NULL,
seek ? &seek : NULL);
} else {
tftp_send_data(client, 1, 0, 0);
}
}
static int handle_reader(struct tftp_client *client)
@@ -405,7 +446,12 @@ static int handle_reader(struct tftp_client *client)
opcode = buf[0] << 8 | buf[1];
if (opcode == OP_ERROR) {
buf[len] = '\0';
printf("[TQFTP] Remote returned an error: %s\n", buf + 4);
int err = buf[2] << 8 | buf[3];
/* "End of Transfer" is not an error, used with stat(2)-like calls */
if (err == ERROR_END_OF_TRANSFER)
printf("[TQFTP] Remote returned END OF TRANSFER: %d - %s\n", err, buf + 4);
else
printf("[TQFTP] Remote returned an error: %d - %s\n", err, buf + 4);
return -1;
} else if (opcode != OP_ACK) {
printf("[TQFTP] Expected ACK, got %d\n", opcode);
@@ -413,19 +459,34 @@ static int handle_reader(struct tftp_client *client)
}
last = buf[2] << 8 | buf[3];
printf("[TQFTP] Got ack for %d\n", last);
// printf("[TQFTP] Got ack for %d\n", last);
/* We've sent enough data for rsize already */
if (last * client->blksize > client->rsize)
return 0;
for (block = last; block < last + client->wsize; block++) {
size_t offset = client->seek + block * client->blksize;
size_t response_size = 0;
/* Check if need to limit response size based for requested rsize */
if ((block + 1) * client->blksize > client->rsize)
response_size = client->rsize % client->blksize;
n = tftp_send_data(client, block + 1,
block * client->blksize);
offset, response_size);
if (n < 0) {
printf("[TQFTP] Sent block %d failed: %zd\n", block + 1, n);
break;
}
printf("[TQFTP] Sent block %d of %zd\n", block + 1, n);
// printf("[TQFTP] Sent block %d of %zd\n", block + 1, n);
if (n == 0)
break;
/* We've sent enough data for rsize already */
if ((block + 1) * client->blksize > client->rsize)
break;
}
return n == 0 ? 0 : 1;
return 1;
}
static int handle_writer(struct tftp_client *client)
@@ -510,6 +571,8 @@ int main(int argc, char **argv)
exit(1);
}
zstd_init();
for (;;) {
FD_ZERO(&rfds);
FD_SET(fd, &rfds);
@@ -571,14 +634,14 @@ int main(int argc, char **argv)
switch (pkt.type) {
case QRTR_TYPE_BYE:
fprintf(stderr, "[TQFTP] got bye\n");
// fprintf(stderr, "[TQFTP] got bye\n");
list_for_each_entry_safe(client, next, &writers, node) {
if (client->sq.sq_node == sq.sq_node)
client_close_and_free(client);
}
break;
case QRTR_TYPE_DEL_CLIENT:
fprintf(stderr, "[TQFTP] got del_client\n");
// fprintf(stderr, "[TQFTP] got del_client\n");
list_for_each_entry_safe(client, next, &writers, node) {
if (!memcmp(&client->sq, &sq, sizeof(sq)))
client_close_and_free(client);
@@ -595,9 +658,13 @@ int main(int argc, char **argv)
handle_rrq(buf, len, &sq);
break;
case OP_WRQ:
printf("[TQFTP] write\n");
// printf("[TQFTP] write\n");
handle_wrq(buf, len, &sq);
break;
case OP_ERROR:
buf[len] = '\0';
printf("[TQFTP] received error: %d - %s\n", buf[2] << 8 | buf[3], buf + 4);
break;
default:
printf("[TQFTP] unhandled op %d\n", opcode);
break;
@@ -607,6 +674,7 @@ int main(int argc, char **argv)
}
close(fd);
zstd_free();
return 0;
}
+12
View File
@@ -0,0 +1,12 @@
[Unit]
Description=QRTR TFTP service
Requires=qrtr-ns.service
After=qrtr-ns.service
[Service]
ExecStart=@prefix@/bin/tqftpserv
Restart=always
[Install]
WantedBy=multi-user.target
+232
View File
@@ -0,0 +1,232 @@
// SPDX-License-Identifier: BSD-3-Clause
/*
* Copyright (c) 2019, Linaro Ltd.
*/
#include <sys/stat.h>
#include <sys/types.h>
#include <dirent.h>
#include <err.h>
#include <errno.h>
#include <fcntl.h>
#include <libgen.h>
#include <limits.h>
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include <unistd.h>
#include "translate.h"
#include "zstd-decompress.h"
#define READONLY_PATH "/readonly/firmware/image/"
#define READWRITE_PATH "/readwrite/"
#ifndef ANDROID
#define FIRMWARE_BASE "/lib/firmware/"
#define TQFTPSERV_TMP "/tmp/tqftpserv"
#else
#define FIRMWARE_BASE "/vendor/firmware/"
#define TQFTPSERV_TMP "/data/vendor/tmp/tqftpserv"
#endif
static int open_maybe_compressed(const char *path);
static void read_fw_path_from_sysfs(char *outbuffer, size_t bufsize)
{
size_t pathsize;
FILE *f = fopen("/sys/module/firmware_class/parameters/path", "rt");
if (!f)
return;
pathsize = fread(outbuffer, sizeof(char), bufsize, f);
fclose(f);
if (pathsize == 0)
return;
/* truncate newline */
outbuffer[pathsize - 1] = '\0';
}
/**
* translate_readonly() - open "file" residing with remoteproc firmware
* @file: file requested, stripped of "/readonly/image/" prefix
*
* It is assumed that the readonly files requested by the client resides under
* /lib/firmware in the same place as its associated remoteproc firmware. This
* function scans through all entries under /sys/class/remoteproc and read the
* dirname of each "firmware" file in an attempt to find, and open(2), the
* requested file.
*
* As these files are readonly, it's not possible to pass flags to open(2).
*
* Return: opened fd on success, -1 otherwise
*/
static int translate_readonly(const char *file)
{
char firmware_value[PATH_MAX];
char *firmware_value_copy = NULL;
char *firmware_path;
char firmware_attr[32];
char path[PATH_MAX];
char fw_sysfs_path[PATH_MAX];
struct dirent *de;
int firmware_fd;
DIR *class_dir;
int class_fd;
ssize_t n;
int fd = -1;
read_fw_path_from_sysfs(fw_sysfs_path, sizeof(fw_sysfs_path));
class_fd = open("/sys/class/remoteproc", O_RDONLY | O_DIRECTORY);
if (class_fd < 0) {
warn("failed to open remoteproc class");
return -1;
}
class_dir = fdopendir(class_fd);
if (!class_dir) {
warn("failed to opendir");
close(class_fd);
return -1;
}
while ((de = readdir(class_dir)) != NULL) {
if (!strcmp(de->d_name, ".") || !strcmp(de->d_name, ".."))
continue;
if (strlen(de->d_name) + sizeof("/firmware") > sizeof(firmware_attr))
continue;
strcpy(firmware_attr, de->d_name);
strcat(firmware_attr, "/firmware");
firmware_fd = openat(class_fd, firmware_attr, O_RDONLY);
if (firmware_fd < 0)
continue;
n = read(firmware_fd, firmware_value, sizeof(firmware_value));
close(firmware_fd);
if (n < 0) {
continue;
}
firmware_value[n] = '\0';
firmware_value_copy = strdup(firmware_value);
firmware_path = dirname(firmware_value_copy);
/* first try path from sysfs */
if ((strlen(fw_sysfs_path) > 0) &&
(strlen(fw_sysfs_path) + 1 + strlen(firmware_value) + 1 + strlen(file) + 1 < sizeof(path))) {
strcpy(path, fw_sysfs_path);
strcat(path, "/");
strcat(path, firmware_path);
strcat(path, "/");
strcat(path, file);
fd = open_maybe_compressed(path);
if (fd >= 0)
break;
if (errno != ENOENT)
warn("failed to open %s", path);
}
/* now try with base path */
if (strlen(FIRMWARE_BASE) + strlen(firmware_value) + 1 +
strlen(file) + 1 > sizeof(path))
continue;
strcpy(path, FIRMWARE_BASE);
strcat(path, firmware_path);
strcat(path, "/");
strcat(path, file);
fd = open_maybe_compressed(path);
if (fd >= 0)
break;
if (errno != ENOENT)
warn("failed to open %s", path);
}
free(firmware_value_copy);
closedir(class_dir);
return fd;
}
/**
* translate_readwrite() - open "file" from a temporary directory
* @file: relative path of the requested file, with /readwrite/ stripped
* @flags: flags to be passed to open(2)
*
* Return: opened fd on success, -1 otherwise
*/
static int translate_readwrite(const char *file, int flags)
{
int base;
int ret;
int fd;
ret = mkdir(TQFTPSERV_TMP, 0700);
if (ret < 0 && errno != EEXIST) {
warn("failed to create temporary tqftpserv directory");
return -1;
}
base = open(TQFTPSERV_TMP, O_RDONLY | O_DIRECTORY);
if (base < 0) {
warn("failed top open temporary tqftpserv directory");
return -1;
}
fd = openat(base, file, flags, 0600);
close(base);
if (fd < 0)
warn("failed to open %s", file);
return fd;
}
/**
* translate_open() - open file after translating path
*
* Strips /readonly/firmware/image and search among remoteproc firmware.
* Replaces /readwrite with a temporary directory.
*/
int translate_open(const char *path, int flags)
{
if (!strncmp(path, READONLY_PATH, strlen(READONLY_PATH)))
return translate_readonly(path + strlen(READONLY_PATH));
else if (!strncmp(path, READWRITE_PATH, strlen(READWRITE_PATH)))
return translate_readwrite(path + strlen(READWRITE_PATH), flags);
fprintf(stderr, "invalid path %s, rejecting\n", path);
errno = ENOENT;
return -1;
}
/* linux-firmware uses .zst as file extension */
#define ZSTD_EXTENSION ".zst"
/**
* open_maybe_compressed() - open a file and maybe decompress it if necessary
* @filename: path to a file that may be compressed (should not include compression format extension)
*
* Return: opened fd on success, -1 on error
*/
static int open_maybe_compressed(const char *path)
{
int fd = -1;
if (access(path, F_OK) == 0) {
fd = open(path, O_RDONLY);
} else {
char *path_with_zstd_extension = NULL;
asprintf(&path_with_zstd_extension, "%s%s", path, ZSTD_EXTENSION);
fd = zstd_decompress_file(path_with_zstd_extension);
free(path_with_zstd_extension);
}
return fd;
}
+6
View File
@@ -0,0 +1,6 @@
#ifndef __TRANSLATE_H__
#define __TRANSLATE_H__
int translate_open(const char *path, int flags);
#endif
+104
View File
@@ -0,0 +1,104 @@
// SPDX-License-Identifier: BSD-3-Clause
/*
* Copyright (c) 2024, Stefan Hansson
*/
/* For memfd_create */
#define _GNU_SOURCE
#include <sys/mman.h>
#include <sys/stat.h>
#include <fcntl.h>
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include <unistd.h>
#include <zstd.h>
#include "zstd-decompress.h"
static ZSTD_DCtx *zstd_context = NULL;
/**
* zstd_init() - set up state for decompression. Needs to be called before zstd_decompress_file()
*/
void zstd_init()
{
zstd_context = ZSTD_createDCtx();
}
/**
* zstd_free() - free state used for decompression. zstd_decompress_file() may not be called after this
*/
void zstd_free()
{
ZSTD_freeDCtx(zstd_context);
}
/**
* zstd_decompress_file() - decompress a zstd-compressed file
* @filename: path to a file to decompress
*
* Return: opened fd on success, -1 on error
*/
int zstd_decompress_file(const char *filename)
{
/* Figure out the size of the file. */
struct stat file_stat;
if (stat(filename, &file_stat) == -1) {
perror("stat failed");
return -1;
}
const size_t file_size = file_stat.st_size;
const int input_file_fd = open(filename, 0);
if (input_file_fd == -1) {
perror("open failed");
return -1;
}
void* const compressed_buffer = mmap(NULL, file_size, PROT_READ, MAP_POPULATE | MAP_PRIVATE, input_file_fd, 0);
if (compressed_buffer == MAP_FAILED) {
perror("mmap failed");
close(input_file_fd);
return -1;
}
close(input_file_fd);
const unsigned long long decompressed_size = ZSTD_getFrameContentSize(compressed_buffer, file_size);
if (decompressed_size == ZSTD_CONTENTSIZE_UNKNOWN) {
fprintf(stderr, "Content size could not be determined for %s\n", filename);
return -1;
}
if (decompressed_size == ZSTD_CONTENTSIZE_ERROR) {
fprintf(stderr, "Error getting content size for %s\n", filename);
return -1;
}
void* const decompressed_buffer = malloc((size_t)decompressed_size);
if (decompressed_buffer == NULL) {
perror("malloc failed");
return -1;
}
const size_t return_size = ZSTD_decompressDCtx(zstd_context, decompressed_buffer, decompressed_size, compressed_buffer, file_size);
if (ZSTD_isError(return_size)) {
fprintf(stderr, "ZSTD_decompress failed: %s\n", ZSTD_getErrorName(return_size));
return -1;
}
const int output_file_fd = memfd_create(filename, 0);
if (output_file_fd == -1) {
perror("memfd_create failed");
return -1;
}
if (write(output_file_fd, decompressed_buffer, decompressed_size) != decompressed_size) {
perror("write failed");
close(output_file_fd);
return -1;
}
return output_file_fd;
}
+15
View File
@@ -0,0 +1,15 @@
// SPDX-License-Identifier: BSD-3-Clause
/*
* Copyright (c) 2024, Stefan Hansson
*/
#ifndef __ZSTD_DECOMPRESS_H__
#define __ZSTD_DECOMPRESS_H__
#include <stdbool.h>
void zstd_init();
void zstd_free();
int zstd_decompress_file(const char *filename);
#endif