Выглядит все хорошо, за исключением тегов aside.
"Для контента, который не является основным предметом статьи (или страницы), но все еще связан со статьей (или страницей), кроме того, что вам нужно, независимо от его визуального оформления."
Я бы сказал, что реклама не связана со статьями. Я бы просто использовал div. Но я думаю, что теоретически реклама может быть связана со статьями, так что это ваш выбор.