Ну, вы не выделили память под результат. То есть ваш result
указывает в случайную точку, и запись в него меняет данные по принципу «на кого бог пошлёт». Отсюда и вылет.
Кроме того, условие while (x!=0)
никогда не наступит (если только с самого начала вы получили не NULL): изменяете-то вы не x
, а *x
! Вы хотели, конечно, крутить цикл пока значение переменной (то есть *x
) не станет равным нулю.
Код (int*)strlen(x)
— грубейшая ошибка. Вы превращаете небольшое число strlen(x)
в указатель, то есть аргумент вашей функции указывает на какую-то чужую память. Хорошо, если это просто вылетит, а не затрёт важные данные.
Теперь остальные ошибки, помельче.
Зачем вы передаёте указатель на аргумент? Во-первых, этим вы исключаете аргумент, являющийся значением выражения (типа p + 1
, то, что называется rvalue
). Во-вторых, вы изменяете переданный аргумент, так что если вам передают число, оно затрётся. Это опасно и не нужно. Передавайте по значению.
Затем, у клиентского кода нет никакого способа узнать, какой же собственно длины переданные ему данные. Не очень хорошо, правда?
Затем, двоичные цифры вы собираете от меньшего разряда к большему. Обычно цифры нужны в обратном порядке. Хотя это может и не быть проблемой, если клиент об этом знает.
А ещё у вас не используется переменная i
.