In NSTableView eigene "Klasse" laden

Christian Kienle · 14. Juni 2006, 19:40

Hallo,

vorhanden:
* zweispaltiges NSTableView (erste Spalte = NSImageCell; zweite Spalte = default)
* eine DataSource (besitzt ein NSMutableArray mit Folder-Objekten)
* eine Klasse "Folder"

Die Klasse Folder hat zwei Eigenschaften:
* image (NSImage)
* name (NSString)

Im - (id)tableView:(NSTableView*)foldersTableView objectValueForTableColumn:(NSTableColumn *) foldersTableViewColumn row:(int)row; Delegate vom NSTableView wähle ich für die entsprechende Zeile das entsprechende Folder-Objekt aus dem NSMutableArray gefüllt mit Folder-Objekten aus. Je nach dem welche Spalte gewünscht wird gebe ich einmal [Folder image] (falls die NSImageCell Spalte Daten möchte) zurück und das andere Mal gebe ich [Folder name] zurück.

Beim Starten des Programmes stürzt es ab - habe die Fehler gesucht - leider nicht gefunden.

Projekt als Zip im Anhang.

Danke.

læng · 14. Juni 2006, 20:09

Quellcode

if([foldersTableViewColumn identifier] == @"FoldersImage")

klappt nicht, denn da werden die Werte der beiden Pointer verglichen. Richtig ist:

Quellcode

if([[foldersTableViewColumn identifier] isEqualToString:@"FoldersImage"])

Christian Kienle · 14. Juni 2006, 20:46

Wow - danke.

gritsch · 14. Juni 2006, 22:47

das kann man sogar noch eleganter lösen wenn man entsprechnde zugriffsmethoden hat bzw die columnIdentifier ordentlich benennt (zb name und image, dann sind das auch schöne methoden-namen)

Quellcode

return [[_folders objectAtIndex:row] performSelector:NSSelectorFromString([foldersTableViewColumn identifier])];

Amin Negm-Awad · 14. Juni 2006, 23:50

Das halte ich alledings für sicherheitsrelevant.

Chris · 15. Juni 2006, 10:45

Solange es Inputmanager und Code Injection gibt stellt das auch keine grössere Gefahr dar.

Chris

gritsch · 15. Juni 2006, 10:46

ich sehe da auch keine große gefahr. gefährlicher wär es sicher wenn man alle Identifier als string angeben und vergleichen würde.

wenn du auf nummer sicher gehen willst dann frag doch zuerst ab obs eine passende methode gibt (respondsToSelector:)

Amin Negm-Awad · 15. Juni 2006, 10:56

Ich vergleiche nicht die Geffahr meines Tuns mit der Gefahr eines anderen Tuns. Es gibt Leute, die an Rosshaaren Schwerter über sich aufhängen. Dennoch würde ich nicht rauchen, weil deren Tun noch gefährlicher ist.

Amin Negm-Awad · 15. Juni 2006, 10:57

Mir ging es nicht um einen Absturz, sondern einen Angriffsverktor. Du holst ja den Methodennamen aus dem Nib. Der ist editierbar.

gritsch · 15. Juni 2006, 11:00

Original von Tom9811
Mir ging es nicht um einen Absturz, sondern einen Angriffsverktor. Du holst ja den Methodennamen aus dem Nib. Der ist editierbar.

und von wem editierbar? Genau: von dir selbst und du wirst wohl wissen was du tust und sonst die fehlermneldung im log lesen und es wieder fixen.
pfuscht jemand anderes im nib rum so ist es seine iegene schuld wenn eine spalte in der table plötzlich leer ist.

wie würdest du es denn machen?

Chris · 15. Juni 2006, 11:10

No risk, no fun.

Amin Negm-Awad · 15. Juni 2006, 11:11

Hehehehehe, dann stecke ich mir erst einmal eine an.

Amin Negm-Awad · 15. Juni 2006, 11:14

Die Nibs sind nicht nur von mir editierbar. Und wie gesagt: Es geht nicht um eine Fehlfunktion (leere Spalte), sondern um einen Angriffsvektor. Ich kann dem Folder-Objekt ja alles mögliche schicken, indem ich den Idenitfier im Nib ändere.

Deine Möglichkeit ist zwar elegant, aber birgt eben diesen Angriffsvektor.

Wie man es anders machen kann, ist ja hier besprochen.

Chris · 15. Juni 2006, 11:32

Das Nib ist ja ein Ordner mit den Dateien classes.nib, info.nib und objects.nib drin.
Wenn du alles bis auf objects.nib löscht dann ist erstmal nix mit editieren.

Chris

Amin Negm-Awad · 15. Juni 2006, 11:43

Na, der Text steht doch drinne. Dann kommt man auch dran. Das Ausnutzen eines Bufferoverflows verlangt da mehr Arbeit.

gritsch · 15. Juni 2006, 11:45

ich find die diskussion inzwischen ja sowas von sinnlos weil wenn jemand ein programm auf seiner platte zum crashen bringen will oder sonstwas dann schafft er das immer. Resourceeditor auf und drauflosmatschen

Amin Negm-Awad · 15. Juni 2006, 11:52

Es geht nicht ums Crashen. Das hatte ich auch bereits angedeutet.

Dem entsprechend geht es auch nicht um die eigene Platte.

Chris · 15. Juni 2006, 11:56

Soviel ist das auch nicht. Heap Exploitation

Chris

gritsch · 15. Juni 2006, 11:57

Aus welchem grund sollte man selbst den identifier änder?
Andere haben an meinem nib nix zu suchen (genau wie an meinen controllern).

Chris · 15. Juni 2006, 11:59

Ich halte es nicht für sinnlos wenn über Sicherheit gesprochen wird.
Ich möchte nicht dass eins meiner Programme für Exploits verwendet werden kann.

Chris