Uni-48541 Fixed warning CA2122 (LinkDemand) (#428)

Author: lassond

Packages/com.unity.formats.fbx/Editor/Scripts/ConvertToModel.cs

@@ -4,6 +4,7 @@
 using System.Linq;
 using UnityEngine.Formats.Fbx.Exporter;
 using System.Runtime.Serialization;
+using System.Security.Permissions;
 
 namespace UnityEditor.Formats.Fbx.Exporter
 {
@@ -103,6 +104,7 @@ public static ExportSettings ExportSettings {
         /// <returns>list of instanced Model Prefabs</returns>
         /// <param name="unityGameObjectsToConvert">Unity game objects to convert to Model Prefab instances</param>
         /// <param name="path">Path to save Model Prefab; use FbxExportSettings if null</param>
+        [SecurityPermission(SecurityAction.LinkDemand)]
         public static GameObject[] CreateInstantiatedModelPrefab (
             GameObject [] unityGameObjectsToConvert)
         {
@@ -165,6 +167,7 @@ public static GameObject[] CreateInstantiatedModelPrefab (
         /// file under a unique filename. May be null, in which case we use
         /// the export settings. Ignored if 'prefabFullPath' is specified.
         /// Ignored if 'toConvert' is a prefab asset.</param>
+        [SecurityPermission(SecurityAction.LinkDemand)]
         public static GameObject Convert (
             GameObject toConvert,
             string fbxDirectoryFullPath = null,
@@ -402,6 +405,7 @@ public static GameObject ApplyOrCreatePrefab(GameObject instance,
         /// </summary>
         /// <param name="toSetUp">Instance in the scene that we want to link to the fbx asset.</param>
         /// <param name="unityMainAsset">Main asset in the FBX.</param>
+        [SecurityPermission(SecurityAction.LinkDemand)]
         public static void SetupFbxPrefab(GameObject toSetUp, GameObject unityMainAsset)
         {
             if(toSetUp == null)

Packages/com.unity.formats.fbx/Editor/Scripts/ConvertToPrefabEditorWindow.cs

@@ -4,6 +4,7 @@
 using UnityEditor.Presets;
 #endif
 using System.Linq;
+using System.Security.Permissions;
 
 namespace UnityEditor.Formats.Fbx.Exporter
 {
@@ -104,6 +105,7 @@ protected bool ExportSetContainsAnimation()
             return false;
         }
 
+        [SecurityPermission(SecurityAction.LinkDemand)]
         protected override bool Export()
         {
             if (string.IsNullOrEmpty(ExportFileName))

Packages/com.unity.formats.fbx/Editor/Scripts/ExportModelEditorWindow.cs

@@ -4,6 +4,7 @@
 using UnityEditor.Presets;
 #endif
 using System.Linq;
+using System.Security.Permissions;
 
 namespace UnityEditor.Formats.Fbx.Exporter
 {
@@ -593,6 +594,7 @@ protected virtual void RestoreSettings()
         }
 
 
+        [SecurityPermission(SecurityAction.LinkDemand)]
         protected override bool Export(){
             if (string.IsNullOrEmpty (ExportFileName)) {
                 Debug.LogError ("FbxExporter: Please specify an fbx filename");

Packages/com.unity.formats.fbx/Editor/Scripts/FbxExportSettings.cs

@@ -32,6 +32,7 @@ public class ExportSettingsEditor : UnityEditor.Editor {
         const float FieldOffset = 18;
         const float BrowseButtonOffset = 5;
 
+        [SecurityPermission(SecurityAction.LinkDemand)]
         public override void OnInspectorGUI() {
             ExportSettings exportSettings = (ExportSettings)target;
 
@@ -310,6 +311,7 @@ private static string GetMayaLocationFromEnvironmentVariable(string env)
         /// <summary>
         /// Returns a set of valid vendor folder paths with no trailing '/'
         /// </summary>
+        [SecurityPermission(SecurityAction.LinkDemand)]
         private static HashSet<string> GetCustomVendorLocations()
         {
             HashSet<string> result = null;
@@ -331,6 +333,7 @@ private static HashSet<string> GetCustomVendorLocations()
             return result;
         }
 
+        [SecurityPermission(SecurityAction.LinkDemand)]
         private static HashSet<string> GetDefaultVendorLocations()
         {
             if (Application.platform == RuntimePlatform.WindowsEditor)
@@ -371,6 +374,7 @@ private static HashSet<string> GetDefaultVendorLocations()
         /// </summary>
         public static List<string> DCCVendorLocations
         {
+            [SecurityPermission(SecurityAction.LinkDemand)]
             get
             {
                 HashSet<string> result = GetCustomVendorLocations();
@@ -743,6 +747,7 @@ private static int FindDCCVersion(string AppName)
         /// 
         /// If MAYA_LOCATION is set, add this to the list as well.
         /// </summary>
+        [SecurityPermission(SecurityAction.LinkDemand)]
         private static void FindDCCInstalls() {
             var dccOptionNames = instance.dccOptionNames;
             var dccOptionPaths = instance.dccOptionPaths;
@@ -806,6 +811,7 @@ private static void FindDCCInstalls() {
         /// <returns>The first valid vendor location</returns>
         public static string FirstValidVendorLocation
         {
+            [SecurityPermission(SecurityAction.LinkDemand)]
             get
             {
                 List<string> locations = DCCVendorLocations;
@@ -849,6 +855,7 @@ private static string GetMayaExePathFromLocation(string location)
             }
         }
 
+        [SecurityPermission(SecurityAction.LinkDemand)]
         public static GUIContent[] GetDCCOptions(){
             if (instance.dccOptionNames == null ||
                 instance.dccOptionNames.Count != instance.dccOptionPaths.Count ||

Packages/com.unity.formats.fbx/Editor/Scripts/FbxExporter.cs

@@ -8,6 +8,7 @@
 using System.Runtime.Serialization;
 using UnityEditor.Formats.Fbx.Exporter.Visitors;
 using UnityEditor.Formats.Fbx.Exporter.CustomExtensions;
+using System.Security.Permissions;
 
 namespace UnityEditor.Formats.Fbx.Exporter
 {
@@ -873,6 +874,7 @@ bool ExportMesh (MeshInfo meshInfo, FbxNode fbxNode)
         /// <summary>
         /// Export GameObject as a skinned mesh with material, bones, a skin and, a bind pose.
         /// </summary>
+        [SecurityPermission(SecurityAction.LinkDemand)]
         protected bool ExportSkinnedMesh (GameObject unityGo, FbxScene fbxScene, FbxNode fbxNode)
         {
             if(!unityGo || fbxNode == null)
@@ -948,6 +950,7 @@ private Matrix4x4 GetBindPose(
         /// Export bones of skinned mesh, if this is a skinned mesh with
         /// bones and bind poses.
         /// </summary>
+        [SecurityPermission(SecurityAction.LinkDemand)]
         private bool ExportSkeleton (SkinnedMeshRenderer skinnedMesh, FbxScene fbxScene, out Dictionary<SkinnedMeshRenderer, Transform[]> skinnedMeshToBonesMap)
         {
             skinnedMeshToBonesMap = new Dictionary<SkinnedMeshRenderer, Transform[]> ();
@@ -1285,6 +1288,7 @@ internal bool ExportTransform (UnityEngine.Transform unityTransform, FbxNode fbx
         /// <summary>
         /// if this game object is a model prefab then export with shared components
         /// </summary>
+        [SecurityPermission(SecurityAction.LinkDemand)]
         protected bool ExportInstance(GameObject unityGo, FbxNode fbxNode)
         {
             if (!unityGo || fbxNode == null)
@@ -1771,6 +1775,7 @@ protected bool ExportConstraints (GameObject unityGo, FbxScene fbxScene, FbxNode
         /// <summary>
         /// Return set of sample times to cover all keys on animation curves
         /// </summary>
+        [SecurityPermission(SecurityAction.LinkDemand)]
         internal static HashSet<float> GetSampleTimes(AnimationCurve[] animCurves, double sampleRate)
         {
             var keyTimes = new HashSet<float>();
@@ -1798,6 +1803,7 @@ internal static HashSet<float> GetSampleTimes(AnimationCurve[] animCurves, doubl
         /// <summary>
         /// Return set of all keys times on animation curves
         /// </summary>
+        [SecurityPermission(SecurityAction.LinkDemand)]
         internal static HashSet<float> GetKeyTimes(AnimationCurve[] animCurves)
         {
             var keyTimes = new HashSet<float>();
@@ -1868,6 +1874,7 @@ internal void ExportAnimationKeys (AnimationCurve uniAnimCurve, FbxAnimCurve fbx
         /// <summary>
         /// Export animation curve key samples
         /// </summary>
+        [SecurityPermission(SecurityAction.LinkDemand)]
         internal void ExportAnimationSamples (AnimationCurve uniAnimCurve, FbxAnimCurve fbxAnimCurve,
             double sampleRate,
             UnityToMayaConvertSceneHelper convertSceneHelper)
@@ -1964,6 +1971,7 @@ protected FbxProperty GetFbxProperty(FbxNode fbxNode, string fbxPropertyName, Sy
         /// NOTE: This is not used for rotations, because we need to convert from
         /// quaternion to euler and various other stuff.
         /// </summary>
+        [SecurityPermission(SecurityAction.LinkDemand)]
         protected void ExportAnimationCurve (FbxNode fbxNode,
                                                 AnimationCurve uniAnimCurve,
                                                 float frameRate,
@@ -2063,6 +2071,7 @@ public float Convert(float value)
         /// <summary>
         /// Export an AnimationClip as a single take
         /// </summary>
+        [SecurityPermission(SecurityAction.LinkDemand)]
         protected void ExportAnimationClip (AnimationClip uniAnimClip, GameObject uniRoot, FbxScene fbxScene)
         {
             if (!uniAnimClip || !uniRoot || fbxScene == null) return;
@@ -2229,6 +2238,7 @@ protected void ExportAnimationClip (AnimationClip uniAnimClip, GameObject uniRoo
         /// <param name="dest">Destination, child of the source.</param>
         /// <param name="sampleRate">Sample rate.</param>
         /// <param name="unityCurves">Unity curves.</param>
+        [SecurityPermission(SecurityAction.LinkDemand)]
         private void TransferMotion(Transform source, Transform dest, float sampleRate, ref Dictionary<GameObject, List<UnityCurve>> unityCurves){
             // get sample times for curves in dest + source
             // at each sample time, evaluate all 18 transfom anim curves, creating 2 transform matrices
@@ -2457,6 +2467,7 @@ ref Dictionary<GameObject, RotationCurve> rotations
         /// <summary>
         /// Export the Animator component on this game object
         /// </summary>
+        [SecurityPermission(SecurityAction.LinkDemand)]
         protected void ExportAnimation (GameObject uniRoot, FbxScene fbxScene)
         {
             if (!uniRoot)
@@ -2641,6 +2652,7 @@ internal int ExportTransformHierarchy(
         ///     but components are only exported if explicitly animated. Meshes are not exported.
         /// </summary>
         /// <returns>The number of nodes exported.</returns>
+        [SecurityPermission(SecurityAction.LinkDemand)]
         internal int ExportAnimationOnly(
             GameObject unityGO,
             FbxScene fbxScene,
@@ -2942,6 +2954,7 @@ private static int GetObjectToRootDepth(Transform startObject, Transform root){
         /// <returns>The animation only hierarchy count.</returns>
         /// <param name="exportSet">GameObject hierarchies selected for export.</param>
         /// <param name="hierarchyToExportData">Map from GameObject hierarchy to animation export data.</param>
+        [SecurityPermission(SecurityAction.LinkDemand)]
         internal int GetAnimOnlyHierarchyCount(Dictionary<GameObject, IExportData> hierarchyToExportData)
         {
             // including any parents of animated objects that are exported
@@ -2960,6 +2973,7 @@ internal int GetAnimOnlyHierarchyCount(Dictionary<GameObject, IExportData> hiera
             return completeExpSet.Count;
         }
 
+        [SecurityPermission(SecurityAction.LinkDemand)]
         internal static Dictionary<GameObject, IExportData> GetExportData(Object[] objects, IExportOptions exportOptions = null)
         {
             if (exportOptions==null)
@@ -2990,6 +3004,7 @@ internal static Dictionary<GameObject, IExportData> GetExportData(Object[] objec
             return exportData.Count == 0 ? null : exportData;
         }
 
+        [SecurityPermission(SecurityAction.LinkDemand)]
         internal static IExportData GetExportData(GameObject rootObject, AnimationClip animationClip, IExportOptions exportOptions = null)
         {
             if (exportOptions==null)
@@ -3090,6 +3105,7 @@ internal static IExportData GetExportData(GameObject go, IExportOptions exportOp
         /// Transform components have already been exported.
         /// This function exports the other components and animation.
         /// </summary>
+        [SecurityPermission(SecurityAction.LinkDemand)]
         protected bool ExportComponents(FbxScene fbxScene, bool exportAnim = true)
         {
             var animationNodes = new HashSet<GameObject> ();
@@ -3186,6 +3202,7 @@ internal int GetHierarchyCount (HashSet<GameObject> exportSet)
         /// </summary>
         /// <returns>The revised export set</returns>
         /// <param name="unityExportSet">Unity export set.</param>
+        [SecurityPermission(SecurityAction.LinkDemand)]
         internal static HashSet<GameObject> RemoveRedundantObjects(IEnumerable<UnityEngine.Object> unityExportSet)
         {
             // basically just remove the descendents from the unity export set
@@ -3296,6 +3313,7 @@ internal enum TransformExportType { Local, Global, Reset };
         ///
         /// This refreshes the asset database.
         /// </summary>
+        [SecurityPermission(SecurityAction.LinkDemand)]
         internal int ExportAll (
             IEnumerable<UnityEngine.Object> unityExportSet, 
             Dictionary<GameObject, IExportData> exportData)
@@ -3927,6 +3945,7 @@ internal static void UnRegisterMeshCallback(GetMeshForObject callback)
         /// This goes through the callback system to find the right mesh and
         /// allow plugins to substitute their own meshes.
         /// </summary>
+        [SecurityPermission(SecurityAction.LinkDemand)]
         bool ExportMesh (GameObject gameObject, FbxNode fbxNode)
         {
             // First allow the object-based callbacks to have a hack at it.
@@ -4060,6 +4079,7 @@ private static void OnExport ()
         }
 
 
+        [SecurityPermission(SecurityAction.LinkDemand)]
         public static string ExportObjects(
             string filePath,
             UnityEngine.Object[] objects = null,
@@ -4073,6 +4093,7 @@ public static string ExportObjects(
         /// Export a list of (Game) objects to FBX file. 
         /// Use the SaveFile panel to allow user to enter a file name.
         /// <summary>
+        [SecurityPermission(SecurityAction.LinkDemand)]
         internal static string ExportObjects (
             string filePath,
             UnityEngine.Object[] objects = null,
@@ -4104,6 +4125,7 @@ internal static string ExportObjects (
             return null;
         }
 
+        [SecurityPermission(SecurityAction.LinkDemand)]
         public static string ExportObject (
             string filePath, 
             UnityEngine.Object root,

Packages/com.unity.formats.fbx/Editor/Scripts/FbxPrefabAutoUpdater.cs

@@ -4,6 +4,7 @@
 using System;
 using System.Runtime.Serialization;
 using UnityEngine.Formats.Fbx.Exporter;
+using System.Security.Permissions;
 
 namespace UnityEditor.Formats.Fbx.Exporter
 {
@@ -98,6 +99,7 @@ public static bool IsPrefabAsset(string assetPath) {
         ///
         /// May return a false positive. This is a cheap check.
         /// </summary>
+        [SecurityPermission(SecurityAction.LinkDemand)]
         public static bool MayHaveFbxPrefabToFbxAsset(string prefabPath,
                 string fbxPrefabScriptPath, HashSet<string> fbxImported) {
             if(fbxImported == null)
@@ -287,6 +289,7 @@ public static bool OnValidateMenuItem()
         /// <summary>
         /// Launch the manual update of the linked prefab specified
         /// </summary>
+        [SecurityPermission(SecurityAction.LinkDemand)]
         public static void UpdateLinkedPrefab(GameObject prefabOrInstance)
         {
             // Find the prefab, bail if this is neither a prefab nor an instance.

Packages/com.unity.formats.fbx/Editor/Scripts/FbxPrefabInspector.cs

@@ -1,11 +1,13 @@
 using UnityEngine;
 using UnityEditor;
 using UnityEngine.Formats.Fbx.Exporter;
+using System.Security.Permissions;
 
 namespace UnityEditor.Formats.Fbx.Exporter {
 
     [CustomEditor(typeof(FbxPrefab))]
     public class FbxPrefabInspector : UnityEditor.Editor {
+        [SecurityPermission(SecurityAction.LinkDemand)]
         public override void OnInspectorGUI() {
 
             SerializedProperty m_GameObjectProp = serializedObject.FindProperty("m_nameMapping");