anotherim/anotherim-desktop
2
0
Fork 0
Code Issues Pull requests Actions Packages Projects Releases Wiki Activity

Fix rare 1 byte buffer over-read

Browse source
This commit is contained in:
Marvin W 2020-06-28 11:25:10 +02:00
parent 48964bc5cc
commit af98b8ea0f
No known key found for this signature in database
GPG key ID: 072E9235DB996F2A
2 changed files with 11 additions and 10 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

19
plugins/gpgme-vala/src/gpgme_helper.vala
View file

@ -144,28 +144,29 @@ private static Key? get_key(string sig, bool priv) throws GLib.Error {
} }
private static string get_string_from_data(Data data) { private static string get_string_from_data(Data data) {
const size_t BUF_SIZE = 256;
data.seek(0); data.seek(0);
uint8[] buf = new uint8[256]; uint8[] buf = new uint8[BUF_SIZE + 1];
ssize_t? len = null; ssize_t len = 0;
string res = ""; string res = "";
do { do {
len = data.read(buf); len = data.read(buf, BUF_SIZE);
if (len > 0) { if (len > 0) {
string part = (string) buf; buf[len] = 0;
part = part.substring(0, (long) len); res += (string) buf;
res += part;
} }
} while (len > 0); } while (len > 0);
return res; return res;
} }
private static uint8[] get_uint8_from_data(Data data) { private static uint8[] get_uint8_from_data(Data data) {
const size_t BUF_SIZE = 256;
data.seek(0); data.seek(0);
uint8[] buf = new uint8[256]; uint8[] buf = new uint8[BUF_SIZE + 1];
ssize_t? len = null; ssize_t len = 0;
ByteArray res = new ByteArray(); ByteArray res = new ByteArray();
do { do {
len = data.read(buf); len = data.read(buf, BUF_SIZE);
if (len > 0) { if (len > 0) {
res.append(buf[0:len]); res.append(buf[0:len]);
} }

2
plugins/gpgme-vala/vapi/gpgme.vapi
View file

@ -474,7 +474,7 @@ namespace GPG {
[CCode (cname = "gpgme_data_release_and_get_mem")] [CCode (cname = "gpgme_data_release_and_get_mem")]
public string release_and_get_mem(out size_t len); public string release_and_get_mem(out size_t len);
public ssize_t read(uint8[] buf); public ssize_t read([CCode (array_length = false)] uint8[] buf, size_t len);
public ssize_t write(uint8[] buf); public ssize_t write(uint8[] buf);