diff options
author | Jan Kalab <pitlicek@gmail.com> | 2010-09-17 08:46:45 +0000 |
---|---|---|
committer | Jan Kalab <pitlicek@gmail.com> | 2010-09-17 08:46:45 +0000 |
commit | 526e8fca14fec8d11be1a3560f1dd907a1c2a5d8 (patch) | |
tree | ba55bc5fef622f8dc6e957689033532a6a7a6b4e /res/res_calendar_ews.c | |
parent | 667f9d28f88e3dc22a086f7bdc6e6bf248bc42ec (diff) |
Merged revisions 287269-287271 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.8
........
r287269 | pitel | 2010-09-17 10:37:49 +0200 (Pá, 17 zář 2010) | 8 lines
Support for HTTP redirects in calendar's URL
libneon does not support HTTP redirects (3xx responses) by default. You must tell it to follow them.
Also, another little unsigned int fix.
(closes issue #17776)
Review: https://reviewboard.asterisk.org/r/921/
........
r287270 | pitel | 2010-09-17 10:42:37 +0200 (Pá, 17 zář 2010) | 6 lines
Asterisk crashing because of double free when EWS request fails
The free is done later in code. I think ast_free() should have built in checks for double free.
(closes issue #17782)
........
r287271 | pitel | 2010-09-17 10:44:28 +0200 (Pá, 17 zář 2010) | 6 lines
Events are visible after they were removed from EWS calendar
Because we must merge calendar even when it's empty.
(closes issue #17786)
........
git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@287272 65c4cc65-6c06-0410-ace0-fbb531ad65f3
Diffstat (limited to 'res/res_calendar_ews.c')
-rw-r--r-- | res/res_calendar_ews.c | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/res/res_calendar_ews.c b/res/res_calendar_ews.c index 504d156f7..31df35312 100644 --- a/res/res_calendar_ews.c +++ b/res/res_calendar_ews.c @@ -35,6 +35,7 @@ ASTERISK_FILE_VERSION(__FILE__, "$Revision$") #include <ne_xml.h> #include <ne_xmlreq.h> #include <ne_utils.h> +#include <ne_redirect.h> #include "asterisk/module.h" #include "asterisk/calendar.h" @@ -197,10 +198,11 @@ static int startelm(void *userdata, int parent, const char *nspace, const char * return NE_XML_ABORT; } - ast_debug(3, "EWS: %d calendar items to load\n", items); + ast_debug(3, "EWS: %u calendar items to load\n", items); ctx->pvt->items = items; if (items < 1) { /* Stop processing XML if there are no events */ + ast_calendar_merge_events(ctx->pvt->owner, ctx->pvt->events); return NE_XML_DECLINE; } return 1; @@ -527,7 +529,6 @@ static int send_ews_request_and_parse(struct ast_str *request, struct xml_contex if (ret != NE_OK) { /* Error handling */ ast_log(LOG_WARNING, "Unable to communicate with Exchange Web Service at '%s': %s\n", ctx->pvt->url, ne_get_error(ctx->pvt->session)); ne_request_destroy(req); - ast_free(request); ne_xml_destroy(parser); return -1; } @@ -838,6 +839,7 @@ static void *ewscal_load_calendar(void *void_data) ast_debug(3, "secret = %s\n", pvt->secret); pvt->session = ne_session_create(pvt->uri.scheme, pvt->uri.host, pvt->uri.port); + ne_redirect_register(pvt->session); ne_set_server_auth(pvt->session, auth_credentials, pvt); ne_set_useragent(pvt->session, "Asterisk"); |