Revision 5400d249
Von Sven Schöling vor etwa 14 Jahren hinzugefügt
| bin/mozilla/admin_printer.pl | ||
|---|---|---|
| 20 | 20 |
} |
| 21 | 21 |
|
| 22 | 22 |
sub printer_dispatcher {
|
| 23 |
$::lxdebug->dump(0, "testing", $::form); |
|
| 24 |
for (qw(get_login_form list_printers add_printer edit_printer save_printer delete_printer list_users)) {
|
|
| 25 |
if ($::form->{$_}) {
|
|
| 26 |
::call_sub($::locale->findsub($_)); |
|
| 23 |
for my $action (qw(get_login_form list_printers add_printer edit_printer save_printer delete_printer list_users)) {
|
|
| 24 |
if ($::form->{$action}) {
|
|
| 25 |
::call_sub($::locale->findsub($action)); |
|
| 27 | 26 |
::end_of_request() |
| 28 | 27 |
} |
| 29 | 28 |
} |
Auch abrufbar als: Unified diff
Printer in Admin - kleine Korrekturen
1. Eine Debugmeldung rausgenommen
2. Eine Unsauberkeit behoben die beim Portieren auf 2.6.1 aufgefallen ist.
Folgender Code crasht:
for (1) {
1 while <>;
}
weil das gealiaste $_ read-only flagged ist, und <> das beschrieben will. Genau
das konnte passieren, wenn die Locales aus auth erstellt wurden und die
special_chars einlesen wollten, aber noch ein alias $_ => 1 aus dem
printer_dispatcher aktuell war.
3. Im admin bereich gezogene user_dbh wurden nicht richtig geschlossen, was ein
Speicherleck unter fcgi bedeutet, und unter cgi das log mit Warnungen flutet,
dass das handle in destruction geschlossen wurde und ein rollback gemacht hat.