plugin-feature: Hash only the actually used feature argument

Clang does not initialize padding in union members so hashing the
complete "arg" union could lead to different hashes if the hashed
plugin_feature_t does not have static storage duration.

Fixes #549.

diff --git a/src/libstrongswan/plugins/plugin_feature.c b/src/libstrongswan/plugins/plugin_feature.c
index 8a1958be5e5093d7cc7532fa37b18fc9f3ab7334..65cdbe9d9f25c374d773163bbd19d0b6720b8c5b 100644 (file)
--- a/src/libstrongswan/plugins/plugin_feature.c
+++ b/src/libstrongswan/plugins/plugin_feature.c
@@ -73,25 +73,55 @@ u_int32_t plugin_feature_hash(plugin_feature_t *feature)
                        data = chunk_empty;
                        break;
                case FEATURE_CRYPTER:
+                       data = chunk_from_thing(feature->arg.crypter);
+                       break;
                case FEATURE_AEAD:
+                       data = chunk_from_thing(feature->arg.aead);
+                       break;
                case FEATURE_SIGNER:
+                       data = chunk_from_thing(feature->arg.signer);
+                       break;
                case FEATURE_HASHER:
+                       data = chunk_from_thing(feature->arg.hasher);
+                       break;
                case FEATURE_PRF:
+                       data = chunk_from_thing(feature->arg.prf);
+                       break;
                case FEATURE_DH:
+                       data = chunk_from_thing(feature->arg.dh_group);
+                       break;
                case FEATURE_PRIVKEY:
+                       data = chunk_from_thing(feature->arg.privkey);
+                       break;
                case FEATURE_PRIVKEY_GEN:
+                       data = chunk_from_thing(feature->arg.privkey_gen);
+                       break;
                case FEATURE_PUBKEY:
+                       data = chunk_from_thing(feature->arg.pubkey);
+                       break;
                case FEATURE_PRIVKEY_SIGN:
+                       data = chunk_from_thing(feature->arg.privkey_sign);
+                       break;
                case FEATURE_PUBKEY_VERIFY:
+                       data = chunk_from_thing(feature->arg.pubkey_verify);
+                       break;
                case FEATURE_PRIVKEY_DECRYPT:
+                       data = chunk_from_thing(feature->arg.privkey_decrypt);
+                       break;
                case FEATURE_PUBKEY_ENCRYPT:
+                       data = chunk_from_thing(feature->arg.pubkey_encrypt);
+                       break;
                case FEATURE_CERT_DECODE:
                case FEATURE_CERT_ENCODE:
+                       data = chunk_from_thing(feature->arg.cert);
+                       break;
                case FEATURE_CONTAINER_DECODE:
                case FEATURE_CONTAINER_ENCODE:
+                       data = chunk_from_thing(feature->arg.container);
+                       break;
                case FEATURE_EAP_SERVER:
                case FEATURE_EAP_PEER:
-                       data = chunk_from_thing(feature->arg);
+                       data = chunk_from_thing(feature->arg.eap);
                        break;
                case FEATURE_CUSTOM:
                        data = chunk_create(feature->arg.custom,