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

Include data forms in entity capabilities hash computation

Browse source
This commit is contained in:
fiaxh 2019-09-02 15:39:06 +02:00
parent 48c2e7afa4
commit ecb3e783b9
3 changed files with 52 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
xmpp-vala/src/module/xep/0004_data_forms.vala
View file

@ -8,6 +8,7 @@ public class DataForm {
public StanzaNode stanza_node { get; set; }
public Gee.List<Field> fields = new ArrayList<Field>();
public string? form_type = null;
public XmppStream stream;
public OnResult on_result;
@ -210,7 +211,12 @@ public class DataForm {
case "fixed":
fields.add(new FixedField(field_node)); break;
case "hidden":
fields.add(new HiddenField.from_node(field_node)); break;
HiddenField field = new HiddenField.from_node(field_node);
if (field.var == "FORM_TYPE") {
this.form_type = field.get_value_string();
break;
}
fields.add(field); break;
case "jid-multi":
fields.add(new JidMultiField(field_node)); break;
case "list-single":

48
xmpp-vala/src/module/xep/0115_entitiy_capabilities.vala
View file

@ -8,6 +8,7 @@ namespace Xmpp.Xep.EntityCapabilities {
private string own_ver_hash;
private Storage storage;
private Regex sha1_base64_regex = /^[A-Za-z0-9+\/]{27}=$/;
public Module(Storage storage) {
this.storage = storage;
@ -15,7 +16,7 @@ namespace Xmpp.Xep.EntityCapabilities {
private string get_own_hash(XmppStream stream) {
if (own_ver_hash == null) {
own_ver_hash = compute_hash(stream.get_module(ServiceDiscovery.Module.IDENTITY).identities, stream.get_flag(ServiceDiscovery.Flag.IDENTITY).features);
own_ver_hash = compute_hash(stream.get_module(ServiceDiscovery.Module.IDENTITY).identities, stream.get_flag(ServiceDiscovery.Flag.IDENTITY).features, new ArrayList<DataForms.DataForm>());
}
return own_ver_hash;
}
@ -47,7 +48,7 @@ namespace Xmpp.Xep.EntityCapabilities {
StanzaNode? c_node = presence.stanza.get_subnode("c", NS_URI);
if (c_node != null) {
string? ver_attribute = c_node.get_attribute("ver", NS_URI);
if (ver_attribute == null) return;
if (ver_attribute == null || !sha1_base64_regex.match(ver_attribute)) return;
Gee.List<string> capabilities = storage.get_features(ver_attribute);
if (capabilities.size == 0) {
stream.get_module(ServiceDiscovery.Module.IDENTITY).request_info(stream, presence.from, (stream, query_result) => {
@ -61,13 +62,19 @@ namespace Xmpp.Xep.EntityCapabilities {
private void store_entity_result(XmppStream stream, string entity, ServiceDiscovery.InfoResult? query_result) {
if (query_result == null) return;
if (compute_hash(query_result.identities, query_result.features) == entity) {
Gee.List<DataForms.DataForm> data_forms = new ArrayList<DataForms.DataForm>();
foreach (StanzaNode node in query_result.iq.stanza.get_deep_subnodes(ServiceDiscovery.NS_URI_INFO + ":query", DataForms.NS_URI + ":x")) {
data_forms.add(DataForms.DataForm.create_from_node(stream, node, (stream, node) => {}));
}
if (compute_hash(query_result.identities, query_result.features, data_forms) == entity) {
storage.store_features(entity, query_result.features);
stream.get_flag(ServiceDiscovery.Flag.IDENTITY).set_entity_features(query_result.iq.from, query_result.features);
}
}
private static string compute_hash(Gee.List<ServiceDiscovery.Identity> identities, Gee.List<string> features) {
private static string compute_hash(Gee.List<ServiceDiscovery.Identity> identities, Gee.List<string> features, Gee.List<DataForms.DataForm> data_forms) {
identities.sort(compare_identities);
features.sort();
@ -82,6 +89,25 @@ namespace Xmpp.Xep.EntityCapabilities {
s += feature + "<";
}
data_forms.sort(compare_data_forms);
foreach (DataForms.DataForm data_form in data_forms) {
if (data_form.form_type == null) {
// If [..] the FORM_TYPE field is not of type "hidden" or the form does not include a FORM_TYPE field, ignore the form but continue processing. (XEP-0115)
continue;
}
s += data_form.form_type + "<";
data_form.fields.sort(compare_data_fields);
foreach (DataForms.DataForm.Field field in data_form.fields) {
s += field.var + "<";
Gee.List<string> values = field.get_values();
values.sort();
foreach (string value in values) {
s += value + "<";
}
}
}
Checksum c = new Checksum(ChecksumType.SHA1);
c.update(s.data, -1);
size_t size = 20;
@ -99,6 +125,20 @@ namespace Xmpp.Xep.EntityCapabilities {
// TODO lang
return 0;
}
private static int compare_data_forms(DataForms.DataForm a, DataForms.DataForm b) {
if (a.form_type != null && b.form_type != null) {
return a.form_type.collate(b.form_type);
}
return 0;
}
private static int compare_data_fields(DataForms.DataForm.Field a, DataForms.DataForm.Field b) {
if (a.var != null && b.var != null) {
return a.var.collate(b.var);
}
return 0;
}
}
public interface Storage : Object {

2
xmpp-vala/src/module/xep/0260_jingle_socks5_bytestreams.vala
View file

@ -470,7 +470,7 @@ class Parameters : Jingle.TransportParameters, Object {
throw new IOError.PROXY_FAILED("wanted %02x for length, got %02x".printf(dstaddr.length, read_buffer[4]));
}
if (!bytes_equal(read_buffer[5:5+dstaddr.length], dstaddr.data)) {
string repr = ((string)read_buffer[5:5+dstaddr.length]).make_valid().escape();
string repr = ((string)read_buffer[5:5+dstaddr.length]).escape(); // TODO call make_valid() once glib>=2.52 becomes widespread
throw new IOError.PROXY_FAILED(@"wanted dstaddr $(dstaddr), got $(repr)");
}
if (read_buffer[5+dstaddr.length] != 0x00 || read_buffer[5+dstaddr.length+1] != 0x00) {