diff options
| author | Nicolas Dufresne <nicolas.dufresne@collabora.co.uk> | 2011-03-22 13:30:16 -0400 |
|---|---|---|
| committer | Nicolas Dufresne <nicolas.dufresne@collabora.co.uk> | 2011-03-25 14:03:03 -0400 |
| commit | 0ebe30fabe59e96f551b1c037441cfc144ccc327 (patch) | |
| tree | 4f84ed202b38c1543304a7ab7d555e1fc6c1e066 | |
| parent | a424d13da48c702da518c117f4b9edff31659e08 (diff) | |
Add call support to log_store_xml_exists()
This implementation is not correct on purpose to ensure a constant time
of execution. See bug #35549 for more details.
| -rw-r--r-- | telepathy-logger/log-store-xml.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/telepathy-logger/log-store-xml.c b/telepathy-logger/log-store-xml.c index cf01b6f..6bdc9f1 100644 --- a/telepathy-logger/log-store-xml.c +++ b/telepathy-logger/log-store-xml.c @@ -671,8 +671,9 @@ log_store_xml_exists (TplLogStore *store, g_return_val_if_fail (TP_IS_ACCOUNT (account), FALSE); g_return_val_if_fail (TPL_IS_ENTITY (target), FALSE); - if (!(type_mask & TPL_EVENT_MASK_TEXT)) - return FALSE; + /* FIXME This method is exposed synchronously in the log manager API and + * thus we need a constant time reply. The implementation is not 100% + * correct here, but provide this constant time. See fd.o but #35549. */ dir = log_store_xml_get_dir (self, account, target); exists = g_file_test (dir, G_FILE_TEST_EXISTS | G_FILE_TEST_IS_DIR); |