Javascriptでスタイルシートを変更するのは、危険なコードになりますか？

下記のコードはセキュリティ上の問題がありますか？もしあるなら、改善策はありますか？
(formタグを使用しているのは、サーバー側で違う内容の処理をさせたいので使用しています。質問の内容に関係無いと思うので省略しました。)

<!DOCTYPE html> <html> <head> (中略) <link rel= "stylesheet" href= "/css/default.css" type= "text/css" id= "link"/> <head> <body> <form action= "" method= "post" name= "myform"> <input type= "radio" name="radio" value= "0" checked />DEFAULT <input type= "radio" name="radio" value= "1" />CUSTOM </form> (中略) </body> <script> var myform = document.myform; myform.addEventListener('change',function(e){ var value = e.target.value; var link = document.getElementById('link'); if(value == 0)link.href= "/css/default.css"; else if(value == 1)link.href= "/css/custom.css"; else alert('error'); }); </script> </html>

m.ts10806

2020/02/09 23:55

コードはマークダウンのcode機能を利用してご提示ください。あと懸念されている「セキュリティ問題」は何でしょうか。「とりあえず見てくれ」だとアドバイスを得るのは難しいと思います。まず自身で調べてください。 IPAのサイトにガイドラインもあります。

hoshi-takanori

2020/02/10 00:24

CSSで見た目を切り替えるだけで、切り替え先のCSSも自分で管理できているならセキュリティ上の問題はないと思います。蛇足ですが、scriptタグをbodyの後ろに書くのは確かHTMLの規格に反するので、bodyの中に書いたほうがいいでしょう。（このままでもほとんどのブラウザで動くとは思いますが。）

thyda.eiqau

2020/02/10 01:39

質問への追記・修正依頼ではありませんが、言語のルールを守ることは非常に重要なことなので、hoshi-takanoriさんのコメントを補足します。 html 要素の中に置いていいコンテンツは 1つの head 要素と、それに続く1つの body 要素と定められています。script 要素はメタデータコンテンツですが、 html 要素は内容にメタデータコンテンツを許容しません。 source: https://html.spec.whatwg.org/multipage/semantics.html#the-html-element