Anruf wegen angeblicher Sicherheitslücke

Thema wurde von Anonymous, 12. März 2014 erstellt.

  1. Anonymous

    Anonymous G-WARD 2015/2016

    Registriert seit:
    20. Februar 2012
    Beiträge:
    8.755
    Danke erhalten:
    1.516
    Danke vergeben:
    1.051
    ich hab da auch 2 Zeilen, mich und "group", was immer das bedeuten soll :)
     
  2. Dennis (MotivMonster.de)

    Dennis (MotivMonster.de) G-WARD 2013/14/15/16

    Registriert seit:
    22. September 2011
    Beiträge:
    30.984
    Danke erhalten:
    6.097
    Danke vergeben:
    1.079
    Beruf:
    Mann für alles :)
    Ort:
    Weilburg
    na da bin ich ja mal gespannt.
    aber generell: Wer mit gefundenen sicherheitslücken nur Kohle machen will ist echt Krimineller abschaum....
     
  3. Dennis (MotivMonster.de)

    Dennis (MotivMonster.de) G-WARD 2013/14/15/16

    Registriert seit:
    22. September 2011
    Beiträge:
    30.984
    Danke erhalten:
    6.097
    Danke vergeben:
    1.079
    Beruf:
    Mann für alles :)
    Ort:
    Weilburg
    ja aber wer halt wirklich sicherheitslücken findet und die ausnutzt um dich im Grunde zu erpressen und oder zu besteheln.....
    das halt kriminell. andere die Lücken finden machen die Lücke bekannt oder melden sie den Programmieren und nutzen die nicht um Kasse zu machen.

    aber egal. mal gespannt was bei rauskommt....
     
  4. Petra

    Petra G-WARD 2013/14/15

    Registriert seit:
    27. August 2011
    Beiträge:
    6.998
    Danke erhalten:
    1.225
    Danke vergeben:
    227
    customer_status beim Admin ist bei mir auch eine 0, und als Admin steht bei customer_added_date das gleiche Format, wie es Manfred im Screen hat (also lauter Nullen)
     
  5. Senior

    Senior Erfahrener Benutzer

    Registriert seit:
    17. April 2013
    Beiträge:
    816
    Danke erhalten:
    178
    Danke vergeben:
    241
    Die vielen Nullen habe ich bei allen Gästen
     
  6. Avenger

    Avenger G-WARD 2012/13/14/15

    Registriert seit:
    26. April 2011
    Beiträge:
    4.771
    Danke erhalten:
    1.478
    Danke vergeben:
    89
    Die admin_access sagt nichts darüber aus, ob der Kunde admin ist.

    Sondern definiert, welche Funktionen ein Admin ausführen darf ("1").
     
  7. Avenger

    Avenger G-WARD 2012/13/14/15

    Registriert seit:
    26. April 2011
    Beiträge:
    4.771
    Danke erhalten:
    1.478
    Danke vergeben:
    89
    Das ist merkwürdig, weil in "create_account.php" mit

    PHP:
    'customers_date_added' => 'now()'
    das Datum auf den aktuellen Wert gesetzt wird.

    In "create_guest_account.php" fehlt dieser Eintrag, so dass die Nullen dort erklärlich sind....

    Du verwendest ja noch eine ältere Version, evtl. fehlt das dort noch....
     
  8. Avenger

    Avenger G-WARD 2012/13/14/15

    Registriert seit:
    26. April 2011
    Beiträge:
    4.771
    Danke erhalten:
    1.478
    Danke vergeben:
    89
    Ich habe den SQL-Injection-Filter noch etwas verbessert, indem ich auch den String "/*" als Anzeichen dafür nehme und dann den Parameter verwerfe.
     

    Anhänge:

  9. Avenger

    Avenger G-WARD 2012/13/14/15

    Registriert seit:
    26. April 2011
    Beiträge:
    4.771
    Danke erhalten:
    1.478
    Danke vergeben:
    89
    Das Loch für die "SQL-Injection" ist jetzt auch gefunden:

    so wie es aussieht, habe ich das eröffnet. :mad:

    Im Vertrauen darauf, dass die diversen Filter solche Dinge entfernen, hatte ich den "GET"-Parameter ohne weitere Filterung verwendet.

    Im Standard-Gambio-Code wird der durch ein "(int)" durchgängig in eine Zahl verwandelt (das SQL also entfernt), so dass es da kein generelles Problem gibt.

    Insofern ist (zumindest bei diesem Angriff) Entwarnung für Gambio GX2 angesagt!
     
  10. Dennis (MotivMonster.de)

    Dennis (MotivMonster.de) G-WARD 2013/14/15/16

    Registriert seit:
    22. September 2011
    Beiträge:
    30.984
    Danke erhalten:
    6.097
    Danke vergeben:
    1.079
    Beruf:
    Mann für alles :)
    Ort:
    Weilburg
    Da wären wir wieder bei den 3. Anbieter Modulen gelandet.
    Fragt sich aber warum jemand mit sowas kasse machen will. kann mir nicht vorstellen das das das erste mal war das er Websites gehackt hat incl. Datenklau zu beweiszwecken.
     
  11. Avenger

    Avenger G-WARD 2012/13/14/15

    Registriert seit:
    26. April 2011
    Beiträge:
    4.771
    Danke erhalten:
    1.478
    Danke vergeben:
    89
    Oder bei einer mangelhaften Filterung...

    Aber ich denke, dass es weniger um "Schuldzuweisungen" gehen sollte, sondern um Absicherung gegen solche und ähnliche Attacken.

    Und gegen nachlässige Programmierer hilft eine zentrale Filterung.

    Auch Du unterlagst ja der (irrigen) Ansicht, dass z.B. der GProtector so was erkennen würde.... :cool:

    http://www.gambio-forum.de/threads/...cherheitslücke?p=114336&viewfull=1#post114336

    Nun, der Herr Nadir hätte durchaus auch massiven Schaden anrichten können.

    z.B. die DB löschen.

    Er ist also noch einer von der freundlicheren Sorte...
     
  12. René (Kids-Inhouse)

    René (Kids-Inhouse) Erfahrener Benutzer

    Registriert seit:
    26. September 2011
    Beiträge:
    344
    Danke erhalten:
    67
    Danke vergeben:
    60
    Guten Morgen!

    Das bedeutet, Grund ist eine spezielle Erweiterung, die nur Achim in seinem Shop im Einsatz hat bzw. hatte? Das ein oder andere Modul habe ich auch aus Avenger's Hand bei mir im Einsatz, z. B. "On the fly" Artikel-Bilder. Da muss ich mir dann aber keine "Sorgen" machen?
     
  13. Avenger

    Avenger G-WARD 2012/13/14/15

    Registriert seit:
    26. April 2011
    Beiträge:
    4.771
    Danke erhalten:
    1.478
    Danke vergeben:
    89
    Nein, es geht nur um die Verwendung des "manufacturers_id" GET-Parameters....
     
  14. Dennis (MotivMonster.de)

    Dennis (MotivMonster.de) G-WARD 2013/14/15/16

    Registriert seit:
    22. September 2011
    Beiträge:
    30.984
    Danke erhalten:
    6.097
    Danke vergeben:
    1.079
    Beruf:
    Mann für alles :)
    Ort:
    Weilburg
    Oh Avenger, das solte keine Schuldzuweisung an dich sein. Sorry - Das war nur weil 3. Anbieter Module eben nicht geprüft werden wie die Aussage von GM eben dieses ja sagte dass 3. Anbieter selbst dafür sorgen müssten. Wer Schuld hat is ja auch kein Thema. Wichtig ist das geprüft wird wie man das generell in Zukunft verhindern kan ohne das System dadurch zu verlangsammen oder zu sehr aufzublähen.
     
  15. Avenger

    Avenger G-WARD 2012/13/14/15

    Registriert seit:
    26. April 2011
    Beiträge:
    4.771
    Danke erhalten:
    1.478
    Danke vergeben:
    89
    Und das halte ich immer noch für ein schlechtes Konzept...

    Es wird ja schon alles mögliche geprüft, im GProtector und im Input-Filter.

    Warum also nicht auf die SQL-Injections?
     
  16. Steffen (indiv-style.de)

    Steffen (indiv-style.de) G-WARD 2013/14/15/16

    Registriert seit:
    30. Juni 2011
    Beiträge:
    5.143
    Danke erhalten:
    1.466
    Danke vergeben:
    452
    Beruf:
    Systemadmin, Webentwickler bei Indiv-Style
    Ort:
    PhpStorm
    Eine verdammt gute Frage!!!
     
  17. Avenger

    Avenger G-WARD 2012/13/14/15

    Registriert seit:
    26. April 2011
    Beiträge:
    4.771
    Danke erhalten:
    1.478
    Danke vergeben:
    89
    Ich habe mir mal den GProtector zu Gemüte geführt, um zu sehen, was der eigentlich tut.

    Die große Überraschung (wenn mich nicht alles täuscht :cool:):

    der untersucht nur Parameter von Programmen aus dem Admin-Bereich.....

    Das ist die Liste der Module, deren Parameter auf Unregelmäßigkeiten geprüft werden

    PHP:
    admin/accounting.php
    admin
    /backup.php
    admin
    /banner_manager.php
    admin
    /banner_statistics.php
    admin
    /blacklist.php
    admin
    /campaigns.php
    admin
    /categories.php
    admin
    /clear_cache.php
    admin
    /configuration.php
    admin
    /content_manager.php
    admin
    /countries.php
    admin
    /coupon_admin.php
    admin
    /coupon_manager.php
    admin
    /create_account.php
    admin
    /cross_sell_groups.php
    admin
    /csv_backend.php
    admin
    /currencies.php
    admin
    /customers.php
    admin
    /customers_status.php
    admin
    /econda.php
    admin
    /ekomi.php
    admin
    /geo_zones.php
    admin
    /gm_backup_files_zip.php.php
    admin
    /gm_ebay.php
    admin
    /gm_emails.php
    admin
    /gm_feature_control.php
    admin
    /gm_gmotion.php
    admin
    /gm_gprint.php
    admin
    /gm_guestbook.php
    admin
    /gm_logo.php
    admin
    /gm_meta.php
    admin
    /gm_miscellaneous.php
    admin
    /gm_module_export.php
    admin
    /gm_module_part_export.php
    admin
    /gm_offline.php
    admin
    /gm_opensearch.php
    admin
    /gm_product_export.php
    admin
    /gm_scroller
    admin
    /gm_security.php
    admin
    /gm_sitemap.php
    admin
    /gm_slider.php
    admin
    /gm_statusbar
    admin
    /gm_trusted_shops_id.php
    admin
    /gm_trusted_shops_widget.php
    admin
    /gv_mail.php
    admin
    /languages.php
    admin
    /lettr_de.php
    admin
    /magnalister.php
    admin
    /mail.php
    admin
    /manufacturers.php
    admin
    /mediafinanz.php
    admin
    /module_export.php
    admin
    /module_newsletter.php
    admin
    /modules.php
    admin
    /nc_clickandbuy.php
    admin
    /orders.php
    admin
    /orders_edit
    admin
    /orders_edit.php
    admin
    /orders_status.php
    admin
    /paypal.php
    admin
    /popup_memo.php
    admin
    /products_attributes.php
    admin
    /products_vpe
    admin
    /products_vpe.php
    admin
    /properties_combis.php
    admin
    /quantity_units.php
    admin
    /shipping_status.php
    admin
    /show_logs.php
    admin
    /specials.php
    admin
    /stats_campaigns.php
    admin
    /stats_sales_report.php
    admin
    /tax_classes.php
    admin
    /tax_rates.php
    admin
    /template_configuration.php
    admin
    /yatego.php
    admin
    /yoochoose.php
    admin
    /zones.php
    D.h., dass es wenig Sinn macht, den im Frontend überhaupt auszuführen (zumindest diese Filterung), kostet eigentlich nur sinnlose Zeit....

    Was evtl. auch an der Stelle Sinn macht ist die Prüfung auf blockierte IPs...

    Wobei die IPs m.E. aber relativ wenig Sinn machen, da normale User sowieso jeden Tag eine neue haben, und die "bösen Buben" sich i.d.R. hinter wechselnden Anonymisierungs-Servern verstecken.
     
  18. Avenger

    Avenger G-WARD 2012/13/14/15

    Registriert seit:
    26. April 2011
    Beiträge:
    4.771
    Danke erhalten:
    1.478
    Danke vergeben:
    89
    Der Input-Filter prüft die Parameter auf das Vorhandendsein einiger HTML-Tags, die evtl das Einschleusen von Javaskripten ermöglichen würden, und entfernt diese falls notwendig.

    PHP:
      var $tagBlacklist = array ('applet''body''bgsound''base''basefont''embed''frame''frameset''head''html''id''iframe''ilayer''layer''link''meta''name''object''script''style''title''xml');
      var 
    $attrBlacklist = array ('action''background''codebase''dynsrc''lowsrc'); // also will strip ALL event handlers
    Wobei der im Admin-Bereich gar nicht angewendet wird....
     
  19. Dennis (MotivMonster.de)

    Dennis (MotivMonster.de) G-WARD 2013/14/15/16

    Registriert seit:
    22. September 2011
    Beiträge:
    30.984
    Danke erhalten:
    6.097
    Danke vergeben:
    1.079
    Beruf:
    Mann für alles :)
    Ort:
    Weilburg
    blockierte IPs ziehelen wohl eher auf die Server ab die für spider und co missbraucht werden. Allerdings sollte man da mal ne Config seite machen das man ne IP für eine bestimmte zeit eintragen kann. schnell und einfach eben.
     
  20. Kai Schoelzke

    Kai Schoelzke Beta-Held

    Registriert seit:
    30. März 2016
    Beiträge:
    3.804
    Danke erhalten:
    548
    Danke vergeben:
    248
    Hallo an Alle,
    ich war gerade bei einer Kundin im Shop habe da folgende Beobachtung gemacht bei wer ist Online.
    Zur Zeit etwa 1200 Besucher Online.
    Über die IP Adresse 176.9.104.38 werden folgende Aufrufe gemacht.
    Code:
    login.php?XTCsid=7b26b0be347eafcde20c1c66421fdf7d%27+AND+(SELECT+8041+FROM(SELECT+COUNT(%2A),CONCAT(0x3a6f79753a,(SELECT+(CASE+WHEN+(8041%3D8041)+THEN+1+ELSE+0+END)),0x3a70687a3a,floor(rand(0)%2A2))x+FROM+INFORMATION_SCHEMA.CHARACTER_SETS+GROUP+BY+x)a)+A
    login.php?XTCsid=7b26b0be347eafcde20c1c66421fdf7d+-6863+union+all+select+1,CONCAT(0x3a6f79753a,0x4244764877697569706b,0x3a70687a3a)%23
    login.php?XTCsid=7ad76c853f72216a6258e0de2eb5a839+%27-6863+union+all+select+CONCAT(0x3a6f79753a,0x4244764877697569706b,0x3a70687a3a),1,1,1,1,1,1,1,1,1%23
    
    Was soll das denn jetzt?