|
 |
8335b1 |
Index: modules/aaa/mod_authz_dbd.c
|
|
|
8335b1 |
===================================================================
|
|
|
8335b1 |
--- a/modules/aaa/mod_authz_dbd.c (revision 1681106)
|
|
|
8335b1 |
+++ b/modules/aaa/mod_authz_dbd.c (revision 1681107)
|
|
|
8335b1 |
@@ -174,7 +174,9 @@
|
|
|
8335b1 |
action, r->user, message?message:noerror);
|
|
|
8335b1 |
}
|
|
|
8335b1 |
else if (newuri == NULL) {
|
|
|
8335b1 |
- newuri = apr_dbd_get_entry(dbd->driver, row, 0);
|
|
|
8335b1 |
+ newuri =
|
|
|
8335b1 |
+ apr_pstrdup(r->pool,
|
|
|
8335b1 |
+ apr_dbd_get_entry(dbd->driver, row, 0));
|
|
|
8335b1 |
}
|
|
|
8335b1 |
/* we can't break out here or row won't get cleaned up */
|
|
|
8335b1 |
}
|
|
|
8335b1 |
@@ -204,7 +206,6 @@
|
|
|
8335b1 |
apr_dbd_prepared_t *query;
|
|
|
8335b1 |
apr_dbd_results_t *res = NULL;
|
|
|
8335b1 |
apr_dbd_row_t *row = NULL;
|
|
|
8335b1 |
- const char **group;
|
|
|
8335b1 |
|
|
|
8335b1 |
if (cfg->query == NULL) {
|
|
|
8335b1 |
ap_log_rerror(APLOG_MARK, APLOG_ERR, 0, r, APLOGNO(01649)
|
|
|
8335b1 |
@@ -224,8 +225,9 @@
|
|
|
8335b1 |
rv != -1;
|
|
|
8335b1 |
rv = apr_dbd_get_row(dbd->driver, r->pool, res, &row, -1)) {
|
|
|
8335b1 |
if (rv == 0) {
|
|
|
8335b1 |
- group = apr_array_push(groups);
|
|
|
8335b1 |
- *group = apr_dbd_get_entry(dbd->driver, row, 0);
|
|
|
8335b1 |
+ APR_ARRAY_PUSH(groups, const char *) =
|
|
|
8335b1 |
+ apr_pstrdup(r->pool,
|
|
|
8335b1 |
+ apr_dbd_get_entry(dbd->driver, row, 0));
|
|
|
8335b1 |
}
|
|
|
8335b1 |
else {
|
|
|
8335b1 |
message = apr_dbd_error(dbd->driver, dbd->handle, rv);
|
|
|
8335b1 |
Index: modules/session/mod_session_dbd.c
|
|
|
8335b1 |
===================================================================
|
|
|
8335b1 |
--- a/modules/session/mod_session_dbd.c (revision 1681106)
|
|
|
8335b1 |
+++ b/modules/session/mod_session_dbd.c (revision 1681107)
|
|
|
8335b1 |
@@ -138,7 +138,8 @@
|
|
|
8335b1 |
return APR_EGENERAL;
|
|
|
8335b1 |
}
|
|
|
8335b1 |
if (*val == NULL) {
|
|
|
8335b1 |
- *val = apr_dbd_get_entry(dbd->driver, row, 0);
|
|
|
8335b1 |
+ *val = apr_pstrdup(r->pool,
|
|
|
8335b1 |
+ apr_dbd_get_entry(dbd->driver, row, 0));
|
|
|
8335b1 |
}
|
|
|
8335b1 |
/* we can't break out here or row won't get cleaned up */
|
|
|
8335b1 |
}
|
|
|
8335b1 |
Index: modules/mappers/mod_rewrite.c
|
|
|
8335b1 |
===================================================================
|
|
|
8335b1 |
--- a/modules/mappers/mod_rewrite.c (revision 1681106)
|
|
|
8335b1 |
+++ b/modules/mappers/mod_rewrite.c (revision 1681107)
|
|
|
8335b1 |
@@ -1352,12 +1352,14 @@
|
|
|
8335b1 |
while ((rv = apr_dbd_get_row(db->driver, r->pool, res, &row, -1)) == 0) {
|
|
|
8335b1 |
++n;
|
|
|
8335b1 |
if (ret == NULL) {
|
|
|
8335b1 |
- ret = apr_dbd_get_entry(db->driver, row, 0);
|
|
|
8335b1 |
+ ret = apr_pstrdup(r->pool,
|
|
|
8335b1 |
+ apr_dbd_get_entry(db->driver, row, 0));
|
|
|
8335b1 |
}
|
|
|
8335b1 |
else {
|
|
|
8335b1 |
/* randomise crudely amongst multiple results */
|
|
|
8335b1 |
if ((double)rand() < (double)RAND_MAX/(double)n) {
|
|
|
8335b1 |
- ret = apr_dbd_get_entry(db->driver, row, 0);
|
|
|
8335b1 |
+ ret = apr_pstrdup(r->pool,
|
|
|
8335b1 |
+ apr_dbd_get_entry(db->driver, row, 0));
|
|
|
8335b1 |
}
|
|
|
8335b1 |
}
|
|
|
8335b1 |
}
|
|
|
8335b1 |
@@ -1370,11 +1372,11 @@
|
|
|
8335b1 |
case 0:
|
|
|
8335b1 |
return NULL;
|
|
|
8335b1 |
case 1:
|
|
|
8335b1 |
- return apr_pstrdup(r->pool, ret);
|
|
|
8335b1 |
+ return ret;
|
|
|
8335b1 |
default:
|
|
|
8335b1 |
/* what's a fair rewritelog level for this? */
|
|
|
8335b1 |
rewritelog((r, 3, NULL, "Multiple values found for %s", key));
|
|
|
8335b1 |
- return apr_pstrdup(r->pool, ret);
|
|
|
8335b1 |
+ return ret;
|
|
|
8335b1 |
}
|
|
|
8335b1 |
}
|
|
|
8335b1 |
|