View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
08252 | Bug reports | ComfortUpdate | public | 2013-10-09 16:01 | 2014-02-16 18:32 |
Reporter | jonsen | Assigned To | c_schmitz | ||
Priority | normal | Severity | minor | ||
Status | closed | Resolution | fixed | ||
Fixed in Version | 2.05+ | ||||
Summary | 08252: The temporary file for PCLZIP is not checked for writability | ||||
Description | The temporary file for the PCLZIP actions in the ComfortUpdate is not checked for writability. Therefore, the ComfortUpdate aborts with an PCLZIP_ERR_WRITE_OPEN_FAIL error, which doesn't clearly state which directory needs to be made writable. Possible solution: The PCLZIP_TEMPORARY_DIR should be checked for writability in step 2 of the ComfortUpdate. | ||||
Additional Information | Example error: http://www.limesurvey.org/en/forum/installation-a-update-issues/93955-error-in-comfort-update Error message: | ||||
Tags | No tags attached. | ||||
Bug heat | 8 | ||||
Complete LimeSurvey version number (& build) | |||||
I will donate to the project if issue is resolved | |||||
Browser | |||||
Database type & version | |||||
Server OS (if known) | |||||
Webserver software & version (if known) | |||||
PHP Version | |||||
A workaround by "dirtyhawk" from the linked thread: chmod -R g+w /path/to/limesurvey |
|
An even better solution would be to have a general path-check, which also checks paths for templates, uploads and the like... EDIT: ...in the admin-panel, which might display problems on login of Super-Admins. |
|
a 'chown apache:apache /path/to/limesurvey' make it work to me (the 'chmod -R g+w /path/to/limesurvey' didn't work for me) |
|
xhiena, your solution does work, as does the chmod variant is most cases. But the problem is that giving apache all-write access is pretty vulnerable for attacks. Thats why it's a workaround and not a solution. Of course, if you system is not "very important", the quick-and-dirty solution is okay, but better is to give apache only the rights it needs to run the site. |
|
from the forum http://www.limesurvey.org/en/forum/installation-a-update-issues/93955-error-in-comfort-update this worked for me as well (this was someones suggestion and it worked) BELOW IS THERE TEXT: Hi! i had exactly this same bug and I managed to avoid this by modifying file: /var/www/limesurvey/application/libraries/admin/pclzip/pclzip.php file. Line: // Samples : I removed // and change tempory dir to /tmp where is 777 rights. define( 'PCLZIP_TEMPORARY_DIR', '/tmp/' ); Reason for this was I wasn't able to find correct place where this zip should have 777 rights by default. No I managed update my limesurvey with comfort update. :woohoo: |
|
Fix committed to master branch: http://bugs.limesurvey.org/plugin.php?page=Source/view&id=13885 |
|
This will be fixed in the next build by using the LS /temp dir. |
|
Fix committed to master branch: http://bugs.limesurvey.org/plugin.php?page=Source/view&id=13888 |
|
Very nice! Is the /temp directory checked for writability though? |
|
During LimeSurvey installation, yes. |
|
2.05+ Build 140216 released |
|
LimeSurvey: master 39e7fb54 2014-02-14 00:49 Details Diff |
Fixed issue 08252: The temporary path for PCLZIP is not properly set |
Affected Issues 08252 |
|
mod - application/controllers/admin/export.php | Diff File | ||
mod - application/controllers/admin/labels.php | Diff File | ||
mod - application/controllers/admin/responses.php | Diff File | ||
mod - application/controllers/admin/surveyadmin.php | Diff File | ||
mod - application/controllers/admin/templates.php | Diff File | ||
mod - application/controllers/admin/update.php | Diff File | ||
add - application/libraries/admin/pclzip.php | Diff File | ||
rm - application/libraries/admin/pclzip/pclerror.lib.php | Diff | ||
rm - application/libraries/admin/pclzip/pcltrace.lib.php | Diff | ||
rm - application/libraries/admin/pclzip/pclzip-trace.lib.php | Diff | ||
rm - application/libraries/admin/pclzip/pclzip.php | Diff | ||
add - application/third_party/pclzip/pclzip.lib.php | Diff File | ||
LimeSurvey: master efa52f53 2014-02-14 13:16 Details Diff |
Fixed issue 08252: The temporary path for PCLZIP is not properly set |
Affected Issues 08252 |
|
mod - application/libraries/admin/pclzip.php | Diff File |
Date Modified | Username | Field | Change |
---|---|---|---|
2013-10-09 16:01 | jonsen | New Issue | |
2013-10-09 16:01 | jonsen | Status | new => assigned |
2013-10-09 16:01 | jonsen | Assigned To | => c_schmitz |
2013-10-10 12:07 | jonsen | Note Added: 26691 | |
2014-01-14 19:39 | jonsen | Note Added: 27866 | |
2014-01-14 19:54 | jonsen | Note Edited: 27866 | |
2014-01-27 18:13 | xhiena | Note Added: 28235 | |
2014-01-28 12:55 | jonsen | Note Added: 28254 | |
2014-02-13 11:53 | durangod | Note Added: 28711 | |
2014-02-13 12:36 | c_schmitz | Project | Feature requests => Bug reports |
2014-02-14 00:52 | c_schmitz | Changeset attached | => LimeSurvey master 39e7fb54 |
2014-02-14 00:52 | c_schmitz | Note Added: 28726 | |
2014-02-14 00:52 | c_schmitz | Resolution | open => fixed |
2014-02-14 00:53 | c_schmitz | Note Added: 28727 | |
2014-02-14 00:53 | c_schmitz | Status | assigned => resolved |
2014-02-14 00:53 | c_schmitz | Fixed in Version | => 2.05+ |
2014-02-14 13:16 | c_schmitz | Changeset attached | => LimeSurvey master efa52f53 |
2014-02-14 13:16 | c_schmitz | Note Added: 28737 | |
2014-02-14 18:46 | jonsen | Note Added: 28745 | |
2014-02-14 21:50 | c_schmitz | Note Added: 28747 | |
2014-02-16 18:32 | c_schmitz | Note Added: 28777 | |
2014-02-16 18:32 | c_schmitz | Status | resolved => closed |