iOSDC 2018 2017年12月2日朝起きたら，Zaim のせいでiPhoneがクラッシュすると Twitter 上で騒がれていました． 前回のリリースから多少は時間がたっていたし，そもそもアプリが iPhone をクラッシュさせるなんてできるの？ 本当にZaimのせいだったのか，そもそも何が起こっていたのか．何が原因だったのか． わかったこと，推測できたこと，喋っても問題なさそうなこと喋ります．

1. iPhoneが数秒おきに
クラッシュするんだけど！
iOSDC 2018
株式会社 Zaim watura (西本 航)

2. 2017年12月02日06時24分

3. iPhoneが数秒おきに
クラッシュしている
という報告が！

4. アプリが数秒おきに
クラッシュしている
ということ？
🤔

5. 違うらしい
iPhoneが再起動するらしい

6. このアプリが悪いらしい

7. 国内最大級750万ダウンロード
突破のオンライン家計簿
結論：Zaimは悪くない

8. 積極採用中
Android Security Marketing
「Zaim スタッフの頭の中」始めました
https://blog.zaim.co.jp
結論：Zaimは悪くない

9. 2017年12月2日
06:24 トレンドになっていることに気がつき で共有

10. 2017年12月2日
06:24 トレンドになっていることに気がつき で共有
06:38
Slack チャンネルを作成
関係者がチャンネルに追加される
06:42 Crashlytics をチェックしていた私Slackに登場

11. @watura (西本航)
•iOS
•インフラ
•セキュリティ
•朝6時から15時まで勤務
•パンを焼いたり
•ジムに行ったり

12. 2017年12月2日
06:24 トレンドになっていることに気がつき で共有
06:38
Slack チャンネルを作成
関係者がチャンネルに追加される
06:42 Crashlytics をチェックしていた私Slackに登場
06:48 原因不明ながら手元で再現

13. 7時頃の時点でわかっていたこと
• 12月2日になったら突然発症
• 一部アプリの通知をオフにすると解消
• 通知を受けない設定だったらクラッシュしない
• Zaimサーバ側にエラーはない
• Xcodeにもエラーは出ない
• Zaim悪者になってる感が感じられる

14. 2017年12月2日
06:24 トレンドになっていることに気がつき Slack で共有
06:38
Slack チャンネルを作成
関係者がチャンネルに追加される
06:42 Crashlytics をチェックしていた私Slackに登場
06:48 原因不明ながら手元で再現
07:40 原因判明

15. 二つのNotification
• Push Notification
• サーバから端末に対して送信
• Local Notification ←こいつが犯人
• 条件を指定してアプリが端末に登録
• どちらを使うにも「通知の許可」が必要
• ユーザ的には違いがない

16. 2017年12月2日
06:24 トレンドになっていることに気がつきSlackで共有
06:38
Slack チャンネルを作成
関係者がチャンネルに追加される
06:42 Crashlytics をチェックしていた私Slackに登場
06:48 原因不明ながら手元で再現
07:40 悪いのはPush通知じゃない
07:51 再現しない版Zaimが手元で作られる

17. やっていたこと
• 発症したユーザを救う方法の探索
• 削除/通知をオフして欲しくない
• 原因の調査
• Zaim上の設定変更で回避したい
→できる
• Twitterで情報共有/収集
• ユーザへの連絡準備

18. 一度発症すると
影響の大きい方法でしか
解消できない。

19. 2017年12月2日
06:24 トレンドになっていることに気がつきSlackで共有
08:02 アプリのアップデートで解消するのは無理と判明
10:40 対応版アップデートをアップロード＆特急申請
10:59
開発者に向けて再現コードの共有
https://github.com/ktakayama/NotificationCrash

20. フルビルド 10 min
↓
サンプルコード 10 sec
努力の結果今は2分くらい

21. 2017年12月2日
06:24 トレンドになっていることに気がつきSlackで共有
10:40
まだ発動していない人のためのアップデートを
アップロード＆特急申請
10:59
開発者に向けて再現コードの共有
https: //github.com/ktakayama/NotificationCrash
11:30 絶対これが原因だろ！というツイートをする人現る

22. https://twitter.com/miyagawa/status/936785405462003713
https://robservatory.com/month-13-is-out-of-bounds/参考

23. 13月は範囲外！？
まじで。。。

24. 2017年12月2日
06:24 トレンドになっていることに気がつきSlackで共有
10:59
開発者に向けて再現コードの共有
https: //github.com/ktakayama/NotificationCrash
11:30 絶対これが原因だろ！というツイートをする人現る
13:30
すでにできることは全部やった感があったので
一旦調査等終了
17:40
iOS 11.2
公式対応記事リリースされる

25. 以降やったこと
• Blog, Twitter等でお知らせ
• 一般ユーザ向け
• https://zaim.co.jp/news/archives/3333
• 技術者向け
• https://zaim.co.jp/news/archives/3337
• お知らせメールの送信
• 訂正依頼
• Zaimが犯人だ！と言っているようなところ

26. ちょっとだけ技術な内容

27. もうあんまり
意味ないんですけどね

28. 対象
iOS 11.0.x
iOS 11.1.x

29. 発見された不具合を起こすコード1
// 2017/12/01のDateComponentsを作る
guard let date = DateComponents(
calendar: Calendar(identifier: .gregorian),
timeZone: TimeZone.autoupdatingCurrent,
year: 2017, month: 12, day: 1).date else { return }
// 毎月1日に実行されるNotificationを作る
let notify = UILocalNotification()
notify.repeatInterval = .month // 毎月繰り返す
notify.fireDate = date // 終了ずみの日付を指定
// Notificationを登録する
UIApplication.shared.scheduleLocalNotification(notify)
// → クラッシュループ
iOS側で実行されるので、iOSがクラッシュする

30. 発見された不具合を起こすコード2
// 日付を指定したDateComponents作成
let dateComponents = DateComponents(day: 1)
// 毎月1日に発動するNotificationのTriggerを作成
let trigger = UNCalendarNotificationTrigger(
dateMatching: dateComponents,
repeats: true)
// 12/01の次にNotificationが発生する日付を計算
print(trigger.nextTriggerDate()) // → メモリを食いつぶす
アプリ側で動いているコードなので
アプリがクラッシュするだけ

31. なぜクラッシュするのか
（推測）
• Local Notificationを管理しているのは、iOS側
• 一定時間ごとに通知すべき通知がないか確認
• 復帰後数秒間は操作できる理由
• 次の繰り返しを登録
• nextTriggerDate()

32. 2017年13月01日 ・・・ NG
2017年14月01日 ・・・ NG
2017年15月01日 ・・・ NG
︙
2017年2147483647月01日 ・・・NG
︙
（推測）

33. なぜZaimが一番燃えたか
• 日本がJST
• 多くのユーザが使っている
• 通知を許可したユーザ全員が対象
• Zaim社は朝一から活動している
• Twitterなどで情報共有した

34. やばい障害が発生した時にやること
1. Blog, Twitter等を調査
2. デモコードを可能な限り作ってテストする
3. 英語で騒ぐ
4. さっさと修正版を申請して、広報する

35. Zaimは悪くない
悪いのは
iOS 11.0, 11.1 だった

36. ご静聴ありがとうございました
注意：
クラッシュとか再起動するとか言ってますが、実際にはSrpingboardがクラッシュしてい
たのでiOSがクラッシュしてた訳ではないと思われます
Android Security Marketing