lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Message-ID: <d212e5a7-e9e5-4297-85fb-030818f7c647@lunn.ch> Date: Fri, 11 Aug 2023 23:01:50 +0200 From: Andrew Lunn <andrew@...n.ch> To: Bjorn Andersson <quic_bjorande@...cinc.com> Cc: Bjorn Andersson <andersson@...nel.org>, Konrad Dybcio <konrad.dybcio@...aro.org>, Chris Lew <quic_clew@...cinc.com>, Alex Elder <elder@...nel.org>, "David S. Miller" <davem@...emloft.net>, Jakub Kicinski <kuba@...nel.org>, Mathieu Poirier <mathieu.poirier@...aro.org>, netdev@...r.kernel.org, linux-kernel@...r.kernel.org, linux-arm-msm@...r.kernel.org, linux-remoteproc@...r.kernel.org Subject: Re: [PATCH v2 2/4] soc: qcom: aoss: Add debugfs interface for sending messages > +static ssize_t qmp_debugfs_write(struct file *file, const char __user *userstr, > + size_t len, loff_t *pos) > +{ > + struct qmp *qmp = file->private_data; > + char buf[QMP_MSG_LEN]; > + int ret; > + > + if (!len || len >= QMP_MSG_LEN) > + return -EINVAL; > + > + if (copy_from_user(buf, userstr, len)) > + return -EFAULT; > + buf[len] = '\0'; > + > + ret = qmp_send(qmp, buf); > + if (ret < 0) > + return ret; Sorry, but you still appear to be sending binary blobs from userspace to the firmware. This is not liked. The documentation you pointed to has three commands. Please implement three debugfs files, one per command. Andrew --- pw-bot: cr
Powered by blists - more mailing lists