[iOS][SDK Version: 3.8.249.1] Crash observed in getLogConfigurationCopy()

Open
#1,353 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Assessment

Difficulty
4/5
Estimated time
3-5 days
Newbie friendliness
25/100
Issue type
Bug
Clarity
Needs clarification
Activity status
Stale
Tech stack
cpp, ios, swift
Domain
mobile-dev, observability-sre

Research direction

Start with the attached crash logs and the stack frames for ODWLogConfiguration.getLogConfigurationCopy(), Telemetry/LogManagerWrapper.swift:60, and TSOneDSTelemetryLogManager.swift:37 and 58. Trace the log creation flow and determine the conditions that make the configuration copy crash; done means the reported crash is understood and the logging path no longer crashes, with coverage if the repository provides a relevant test location.

Written by the indexing model from the issue text.

Description

bug

Describe your environment. Describe any aspect of your environment relevant to the problem, including your SDK version, platform, OS version, etc. If you're reporting a problem with a specific version of a library in this repo, please check whether the problem has been fixed on main brach.

Steps to reproduce.
We dont have a specific step to reproduce the crash. Crash points to log creation flow.

What is the expected behavior?
What did you expect to see?
There should not be any crash happening.

Additional context.
Attaching the crash log here.
0 TeamsCommon 0x000000010ea01bf0 ZNSt3__13mapINS_12basic_stringIcNS_11char_traitsIcEENS_9allocatorIcEEEEN9Microsoft12Applications6Events7VariantENS_4lessIS6_EENS4_INS_4pairIKS6_SA_EEEEE6insertB8ne180100INS_20__map_const_iteratorINS_21__tree_const_iteratorINS_12__value_typeIS6_SA_EEPNS_11__tree_nodeISM_PvEElEEEEEEvT_ST (__tree:112)
1 TeamsCommon 0x000000010ea01b70 ZNSt3__13mapINS_12basic_stringIcNS_11char_traitsIcEENS_9allocatorIcEEEEN9Microsoft12Applications6Events7VariantENS_4lessIS6_EENS4_INS_4pairIKS6_SA_EEEEEC2B8ne180100ERKSH (map:1078)
2 TeamsCommon 0x000000010ea010b0 +[ODWLogConfiguration getLogConfigurationCopy] (map:1078)
3 TeamsCommon 0x000000010f094acc Telemetry.ODWLogManagerWrapper.loggerForTenant(: Swift.String, configuration: [Swift.String : Swift.String]) -> Telemetry.TSTelemetryLoggerProtocol? (LogManagerWrapper.swift:60)_
4 TeamsCommon 0x000000010f094f18 protocol witness for Telemetry.LogManagerProtocol.loggerForTenant(_: Swift.String, configuration: [Swift.String : Swift.String]) -> Telemetry.TSTelemetryLoggerProtocol? in conformance Telemetry.ODWLogManagerWrapper : Telemetry.LogManagerProtocol in Telemetry (:0)
5 TeamsCommon 0x000000010f0ad974 Telemetry.TSOneDSTelemetryLogManager.(logger in _0254B3F70CF7F7D778C8FF7F5624B841)(for: Swift.String, configuration: [Swift.String : Swift.String]) -> Telemetry.TSTelemetryLoggerProtocol (TSOneDSTelemetryLogManager.swift:58)
6 TeamsCommon 0x000000010f0ad474 Telemetry.TSOneDSTelemetryLogManager.logger(for: Swift.String, collectorUri: Swift.String) -> Telemetry.TSTelemetryLoggerProtocol? (TSOneDSTelemetryLogManager.swift:37)
7 TeamsCommon 0x000000010f0ad9fc @objc Telemetry.TSOneDSTelemetryLogManager.logger(for: Swift.String, collectorUri: Swift.String) -> Telemetry.TSTelemetryLoggerProtocol? (:0)

Image

report-2516521581180009999-45b5786e-7336-4fc6-97ec-09fb23b6cc1c.txt
report-2516521610430009999-1d7914ab-a2d3-4565-84a5-bfba859063cb.txt

Dominant language
C
Stars
102
Forks
66
Avg merge
3d 3h
Merged PRs (30d)
11

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from microsoft/cpp_client_telemetry

All issues in microsoft/cpp_client_telemetry

Similar issues

More C issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.