Format string is not a string literal (potentially insecure)

macuser · 11. März 2014, 14:45

Seit heute mit dem neuen 5.1 update von Xcode bekommen ich bei folgendem code die warning
"Format string is not a string literal (potentially insecure)"

Quellcode

NSRunAlertPanel(@"MySQL Server", [NSString stringWithFormat:@"Der letzte Datenbankquery brachte einen Fehler: %@",[self.mySQLConnection lastErrorMessage]], @"Ok", nil, nil);

[/i]
[/i]die Warnung bezieht sich auf [NSString stringWithFormat:@"Der letzte Datenbankquery brachte einen Fehler: %@",[self.mySQLConnection lastErrorMessage]]
aber warum ist das seit dem update falsch?

Michael · 11. März 2014, 14:57

Was gibt denn die Methode mySQLConnection für einen Datentyp zurpück?

macuser · 11. März 2014, 17:06

lastErrorMessage ist ein NSString

Thallius · 11. März 2014, 17:15

Ich würde das Debuggen indem ich das komplett zerlege in

Quellcode

NSString *error=[self.mySQLConnection lastErrorMessage];
NSString *message=[NSString stringWithFormat:@"....... %@",error];
NSRunAlertPanel(@"MySQL Server", message, @"Ok", nil, nil);

Dann kannst du breakpoints setzen und dir die Inhalte der einzelnen Variablen ansehen

Gruß

Claus

macuser · 11. März 2014, 17:27

hab ich grad getan und es ist alles in Ordnung. lastErrorMessage gibt mir einen nsstring zurück.
diese Warnung bekomm ich auch erst seit heute nach dem update

Thallius · 11. März 2014, 17:36

WEnn du es genauso machst wie ich, dann kommt der Fehler immer noch in der Zeile wo die message zusammen gebaut wird?

Gruß

Claus

Chris · 11. März 2014, 18:11

macuser schrieb:

Seit heute mit dem neuen 5.1 update von Xcode bekommen ich bei folgendem code die warning
"Format string is not a string literal (potentially insecure)"

Quellcode

NSRunAlertPanel(@"MySQL Server", [NSString stringWithFormat:@"Der letzte Datenbankquery brachte einen Fehler: %@",[self.mySQLConnection lastErrorMessage]], @"Ok", nil, nil);

[/i]
[/i]die Warnung bezieht sich auf [NSString stringWithFormat:@"Der letzte Datenbankquery brachte einen Fehler: %@",[self.mySQLConnection lastErrorMessage]]
aber warum ist das seit dem update falsch?

Das geht ja auch so:

Quellcode

NSRunAlertPanel(@"MySQL Server", @"Der letzte Datenbankquery brachte einen Fehler: %@", @"Ok", nil, nil, [self.mySQLConnection lastErrorMessage]);

Der 2. Parameter heisst nicht umsonst msgFormat.

Chris

Thallius · 11. März 2014, 19:11

Wow das ist aber auch eine ziemlich kranke syntax...

Zum Glück deprecated

Gruß

Claus

gritsch · 11. März 2014, 20:15

das war doch immer schon so dass die ans ende kommen. ist ja auch aus der doku ersichtlich und macht auch nur so sinn.

macuser · 12. März 2014, 15:57

jetzt funktioniert es. ich habe bisher nie auf den parameternamen geachtet.
aber die Syntax ist es wirklich etwas strange

danke auf jeden fall

gritsch · 12. März 2014, 16:26

macuser schrieb:

jetzt funktioniert es. ich habe bisher nie auf den parameternamen geachtet.
aber die Syntax ist es wirklich etwas strange

danke auf jeden fall

was soll daran strange sein?
ist doch der einzig logische weg!

macuser · 12. März 2014, 16:31

ja mag sein, aber es ist nicht die formatedString Syntax die man von NSString kennt. deshalb strange

gritsch · 12. März 2014, 16:33

macuser schrieb:

ja mag sein, aber es ist nicht die formatedString Syntax die man von NSString kennt. deshalb strange

doch natürlich. genau die ist es.

nur dass du dort keine FIXE parameter hast

Thallius · 12. März 2014, 19:20

gritsch schrieb:

macuser schrieb:

ja mag sein, aber es ist nicht die formatedString Syntax die man von NSString kennt. deshalb strange

doch natürlich. genau die ist es.

nur dass du dort keine FIXE parameter hast

Wo liegt denn der Sinn dort einen Parameter nur für das Format zu machen und die Parameter selber am Ende anzufügen? Den Weg dort einfach den kompletten Text als NSString statt nur dem Format anzugeben wäre wesentlich logischer und auch einfacher zu handhaben.

Gruß

Claus

gritsch · 12. März 2014, 21:31

wenn dir die syntax nicht passt, dann nimm doch NSAlert.
ist meiner meinung nach halt viel unübersichtlicher statt einem einzeiler einen riesen wulst an code zu haben - aber jedem das seine

Amin Negm-Awad · 18. März 2014, 12:53

Thallius schrieb:

gritsch schrieb:

macuser schrieb:

ja mag sein, aber es ist nicht die formatedString Syntax die man von NSString kennt. deshalb strange

doch natürlich. genau die ist es.

nur dass du dort keine FIXE parameter hast

Wo liegt denn der Sinn dort einen Parameter nur für das Format zu machen und die Parameter selber am Ende anzufügen? Den Weg dort einfach den kompletten Text als NSString statt nur dem Format anzugeben wäre wesentlich logischer und auch einfacher zu handhaben.

Gruß

Claus

Dann willst du auch, dass NSLog() nur den fertigen String nimmt, weil man ja den kompletten String vorab machen kann und ihn dann fix und fertig übergibt?

Quellcode

Quellcode

Quellcode

Quellcode

Ähnliche Themen