`paragraphRange` signature does not specify the returning type could be nullable, leading to unexpected crashes
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 2/5
- Tempo stimato
- 1-3 ore
- Idoneità per principianti
- 35/100
- Tipo di issue
- Bug
- Chiarezza
- Abbastanza chiara
- Stato di attività
- Ferma
- Stack tecnologico
- swift
- Ambito
- mobile-dev
Direzione di ricerca
Inizia in Aztec/Classes/Extensions/NSAttributedString+ParagraphRange.swift, in paragraphRange(for: NSTextAttachment), soprattutto nella chiamata a ranges(forAttachment:). Cerca i suoi chiamanti per capire come viene utilizzato il valore restituito, quindi verifica che il percorso in cui l’allegato non viene trovato non provochi più un arresto anomalo imprevisto e che i chiamanti interessati gestiscano il risultato in modo appropriato.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
We've seen some crashes in WooCommerce for iOS coming from our AztecEditor wrapper. This seems to come from the underlying usage of paragraphRange, since assumes that the attachment passed into the method is the string, force-unwrapping the first result of the attachments array:
func paragraphRange(for attachment: NSTextAttachment) -> NSRange {
// We assume the attachment IS in the string. This method should not be called otherwise.
let attachmentRange = ranges(forAttachment: attachment).first!
return paragraphRange(for: attachmentRange)
}
This leads to unexpected crashes when the method is not used as intended, since its signature does not specify that the result could be nil. Ideally we should change the returning type to NSRange?, or handle the existence of nil values internally.
- Lingua principale
- Swift
- Stelle
- 667
- Fork
- 153
- Merge medio
- 2g 6h
- PR unite (30g)
- 4
Preparare l'ambiente
- Nessun Dockerfile né file Docker Compose
- Ha un modello di pull request
- Leggi la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di wordpress-mobile/AztecEditor-iOS
-
Bug
Difficoltà 3/5 1-2 giorni Idoneità per principianti 45/100
wordpress-mobile/AztecEditor-iOS#1417 · 1 commento ·
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 45/100
wordpress-mobile/AztecEditor-iOS#1411 ·
-
iOS 17 Crash - UIGraphicsBeginImageContext() failed to allocate CGBitampContextForse di nuovo libera @jkmassel l’ha presa 619 giorni fa e non c’è nessuna pull request aperta. Aperta
wordpress-mobile/AztecEditor-iOS#1401 · 2 commenti · 1 assegnatario ·
-
Difficoltà 3/5 1-2 giorni Idoneità per principianti 35/100
wordpress-mobile/AztecEditor-iOS#1399 ·
-
Difficoltà 3/5 1-2 giorni Idoneità per principianti 35/100
wordpress-mobile/AztecEditor-iOS#1398 · 1 commento · 1 reazione ·
Tutte le issue di wordpress-mobile/AztecEditor-iOS
Issue simili
-
bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 64/100
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 70/100
xiufengsun/TokenTracker#750 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
product / databases product / functions product / self-hosted product / storage sdk / cli
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
I maintainer di solito rispondono entro 1 giorno
-
area:ai-suggestions bug P0 security
Difficoltà 2/5 1-3 ore Idoneità per principianti 84/100
uttrflow/uttrflow-swift#5182 ·
I maintainer di solito rispondono entro 1 giorno
-
bug bug:behavior clawsweeper:needs-maintainer-review clawsweeper:needs-product-decision clawsweeper:no-new-fix-pr clawsweeper:source-repro impact:ux-friction issue-rating: 🦞 diamond lobster P3
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
openclaw/openclaw#164903 · 1 commento · 1 reazione ·
I maintainer di solito rispondono entro 1 giorno