summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorQuentin Rameau <quinq@fifth.space>2017-05-18 12:33:28 +0200
committerQuentin Rameau <quinq@fifth.space>2017-05-22 12:38:49 +0200
commit2223417c91569f8314205d8f34c3cdf4d96d6e1a (patch)
tree80ff68618203dcc2677ebdcfa4eaee4eaef38b89
parenta1328457cff30dc678a1b8e80dc44ddb73ce293c (diff)
Fix handling of uri-specific parameters
We need to (re)apply uri-specific parameters for each new uri even if the parameter has already been set to get the correct value. Thanks to Julien STEINHAUSER <julien.steinhauser@orange.fr> for the report!
-rw-r--r--surf.c28
1 files changed, 19 insertions, 9 deletions
diff --git a/surf.c b/surf.c
index 51c1813..7e1bdcb 100644
--- a/surf.c
+++ b/surf.c
@@ -643,7 +643,7 @@ cookiepolicy_set(const WebKitCookieAcceptPolicy p)
void
seturiparameters(Client *c, const char *uri)
{
- Parameter *newconfig = NULL;
+ Parameter *config, *newconfig = NULL;
int i;
for (i = 0; i < LENGTH(uriparams); ++i) {
@@ -656,16 +656,26 @@ seturiparameters(Client *c, const char *uri)
if (!newconfig)
newconfig = defconfig;
- if (newconfig == curconfig)
- return;
for (i = 0; i < ParameterLast; ++i) {
- if (defconfig[i].force)
- continue;
- if (newconfig[i].force)
- setparameter(c, 0, i, &newconfig[i].val);
- else if (curconfig[i].force)
- setparameter(c, 0, i, &defconfig[i].val);
+ switch(i) {
+ case Certificate:
+ case CookiePolicies:
+ case Style:
+ config = defconfig[i].force ? defconfig :
+ newconfig[i].force ? newconfig :
+ defconfig;
+ break;
+ default:
+ if (newconfig == curconfig || defconfig[i].force)
+ continue;
+ config = newconfig[i].force ? newconfig :
+ curconfig[i].force ? defconfig :
+ NULL;
+ }
+
+ if (config)
+ setparameter(c, 0, i, &config[i].val);
}
curconfig = newconfig;