Bug 28102 - fehlerhafte Überprüfung auf request.options typ in umc-server response
fehlerhafte Überprüfung auf request.options typ in umc-server response
Status: CLOSED FIXED
Product: UCS
Classification: Unclassified
Component: UMC (Generic)
UCS 3.0
Other Linux
: P3 normal (vote)
: UCS 3.0-2-errata
Assigned To: Alexander Kläser
Jascha Geerds
:
Depends on:
Blocks: 28182
  Show dependency treegraph
 
Reported: 2012-07-27 15:59 CEST by Florian Best
Modified: 2012-08-15 13:36 CEST (History)
1 user (show)

See Also:
What kind of report is it?: ---
What type of bug is this?: ---
Who will be affected by this bug?: ---
How will those affected feel about the bug?: ---
User Pain:
Enterprise Customer affected?:
School Customer affected?:
ISV affected?:
Waiting Support:
Flags outvoted (downgraded) after PO Review:
Ticket number:
Bug group (optional):
Max CVSS v3 score:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Florian Best univentionstaff 2012-07-27 15:59:58 CEST
Auswirkung von Bug #26635

{ "status": "591 ", "message": "Die Ausf�hrung des Kommandos computerroom/vnc ist fehlgeschlagen: Traceback (most recent call last): File '/usr/lib/pymodules/python2.6/univention/management/console/modules/__init__.py', line 195, in execute func( request ) File '/usr/lib/pymodules/python2.6/univention/management/console/modules/computerroom/__init__.py', line 413, in vnc self.finished( request.id, response ) File '/usr/lib/pymodules/python2.6/univention/management/console/modules/__init__.py', line 253, in finished self.result( res ) File '/usr/lib/pymodules/python2.6/univention/management/console/modules/__init__.py', line 263, in result self.signal_emit( 'failure', response ) File '/usr/lib/pymodules/python2.6/notifier/signals.py', line 75, in signal_emit self.__signals[ signal ].emit( *args ) File '/usr/lib/pymodules/python2.6/notifier/signals.py', line 41, in emit cb( *args ) File '/usr/lib/pymodules/python2.6/notifier/__init__.py', line 104, in __call__ return self._function( *tmp, **self._kwargs ) File '/usr/lib/pymodules/python2.6/univention/management/console/protocol/modserver.py', line 109, in _reply self.response( msg ) File '/usr/lib/pymodules/python2.6/univention/management/console/protocol/modserver.py', line 292, in response data = str( msg ) File '/usr/lib/pymodules/python2.6/univention/management/console/protocol/message.py', line 311, in __str__ del body['options'] TypeError: 'str' object does not support item deletion " }
Comment 1 Alexander Kläser univentionstaff 2012-07-27 16:03:57 CEST
Hier sollte zunächst body darauf überprüft werden, ob der Typ dict ist.
Comment 2 Alexander Kläser univentionstaff 2012-07-27 16:06:09 CEST
Dies kann zusammen mit Bug 26490 gelöst werden.
Comment 3 Florian Best univentionstaff 2012-07-28 14:47:38 CEST
Kann der patch aus Bug #27875 hier mit eingespielt werden? Finde ich kritisch, (Gerade in UCS@school kann das sehr viel Ärger bereiten).
Comment 4 Alexander Kläser univentionstaff 2012-07-30 16:06:23 CEST
Der Patch wurde für das Errata eingespielt und die YAML-Datei 2012-07-24-univention-management-console.yaml wurde entsprechend eingepasst.

Die Änderung wurde auch in 3.1 eingespielt:
====================
 univention-management-console (5.0.3-1) unstable; urgency=low
 .
   * a missing type check has been fixed in the Response object which could
     cause a traceback in rare cases; Bug #28102
====================

(In reply to comment #3)
> Kann der patch aus Bug #27875 hier mit eingespielt werden? Finde ich kritisch,
> (Gerade in UCS@school kann das sehr viel Ärger bereiten).

Guter Hinweis. Das Problem wird über diesen Bug geregelt, habe ihn auf errata3.0-2 getaggt.

Die Pakete (Errata + 3.1) sind gebaut.
Comment 5 Alexander Kläser univentionstaff 2012-07-30 16:06:51 CEST
(In reply to comment #4)
> ...
> (In reply to comment #3)
> > Kann der patch aus Bug #27875 hier mit eingespielt werden? Finde ich kritisch,
> > (Gerade in UCS@school kann das sehr viel Ärger bereiten).
> 
> Guter Hinweis. Das Problem wird über diesen Bug geregelt, habe ihn auf
> errata3.0-2 getaggt.

→ "diesen Bug" will heißen Bug 27875
Comment 6 Alexander Kläser univentionstaff 2012-07-30 16:51:55 CEST
Changelog-Eintrag wurde hinzugefügt.
Comment 7 Jascha Geerds univentionstaff 2012-07-31 09:50:29 CEST
- Bug konnte reproduziert werde
- Der Fix funktioniert
- YAML ist in Ordnung
- Änderungen nur unter 3.0-2 einspielbar

- Fix in UCS 3.1 übernommen
- Version vom Sourcepaket wurde erhöht
- Eintrag im Changelog vorhanden
Comment 8 Alexander Kläser univentionstaff 2012-07-31 11:58:59 CEST
Das Errata-Update ist raus.
Comment 9 Jascha Geerds univentionstaff 2012-07-31 12:05:44 CEST
Verified!